PHP Forum

**Halbarad** · 01:41 PM

Hi,

i am trying to learn PHP and no i have managed to create member system with memberlist with help. And now i want to add a function that allows members to edit their information.

I tried it with this code but it does not work at all:
please help me out a bit!

PHP Code:


			
<?php

session_start(); 



if (!empty($_POST[username]))

{

    if ($_POST[password] != $_POST[password2])

        exit("Error - Passwords don't match. Please go back and try again.");



    require_once("connect.php");



    $query = mysql_query("update set members 

    (province, kingdom, email, msn, password)

    $affected_rows = $sql->id = $_SESSION[id];

    VALUES    (,'$_POST[province]','$_POST[kingdom]','$_POST[email]','$_POST[msn]','$_POST[password]')")



    or die ("Error - Couldn't edit account.");

    

 include("header.inc"); 

    echo "You have succesfully edited your account!<br /><br />

        Please login <a href='login.php'><b>here</b></a>.";

 include("footer.inc"); 

    exit();

}



?>





<?php include("header.inc"); ?>        <form action="account.php" method="post">

            <table width="75%" align="center" cellpadding="3" cellspacing="1" style="border: 5px solid #FFFFFF">

                <tr>

                    <td width="100%" style="font-family: verdana; font-size: 10px"><b>edit your account</b></td>

                </tr>

                <tr>

                    <td width="100%" style="font-family: verdana; font-size: 10px"><label>Province Name: <br /><input type="text" name="province" size="20" value="<? echo $_POST[province]; ?>"></label></td>

                </tr>

                <tr>

                    <td width="100%" style="font-family: verdana; font-size: 10px"><label>Kingdom Coordinates: <br /><input type="text" name="kingdom" size="20" value="<? echo $_POST[kingdom]; ?>"></label></td>

                </tr>

                <tr>

                    <td width="100%" style="font-family: verdana; font-size: 10px"><label>Email: <br /><input type="text" name="email" size="20" value="<? echo $_POST[email]; ?>"></label></td>

                </tr>

                <tr>

                    <td width="100%" style="font-family: verdana; font-size: 10px"><label>MSN: <br /><input type="text" name="msn" size="20" value="<? echo $_POST[msn]; ?>"></label></td>

                </tr>

                <tr>

                    <td width="100%" style="font-family: verdana; font-size: 10px"><label>Password: <br /><input type="password" name="password" size="20" value="<? echo $_POST[password]; ?>"></label></td>

                </tr>

                <tr>

                    <td width="100%" style="font-family: verdana; font-size: 10px"><label>Verify Password: <br /><input type="password" name="password2" size="20" value=""></label></td>

                </tr>

                <tr>

                    <td width="100%" style="font-family: verdana; font-size: 10px"><input type="submit" value="Edit!"></td>

                </tr>

            </table>

        </form>

<?php include("footer.inc"); ?>

**Halbarad** · 09-13-2005, 03:49 PM

**nasaboy007** · 09-13-2005, 04:51 PM

whats the error thats returned?

i see that u need single quotes around some of ur POST variables. like it should be $_POST['username']

**Halbarad** · 09-13-2005, 05:03 PM

its doesn't return any error, it just doesnt update the database

**AxE** · 09-13-2005, 05:10 PM

PHP Code:


			
  $query = mysql_query("update set members

    (province, kingdom, email, msn, password)

    $affected_rows = $sql->id = $_SESSION[id];

    VALUES    (,'$_POST[province]','$_POST[kingdom]','$_POST[email]','$_POST[msn]','$_POST[password]')")

Try

PHP Code:


			
  $query = mysql_query("UPDATE `members`

SET `province` = '".$_POST['province']."',`kingdom` = '".$_POST['kingdom']."',`email` = '".$_POST['email']."',`msn` = '".$_POST['msn']."',`password` = '".$_POST['password']."'

WHERE `id`='".$_SESSION['id']."'");

I think the WHERE part is right, though not certain. The other part should work.

**Halbarad** · 04:53 PM

nope it still doesnt work

@ AxE: when i try your option, i get this message:

Parse error: parse error, unexpected T_LOGICAL_OR in /home/halbarad/public_html/ai/account.php on line 17

**AxE** · 09-15-2005, 01:13 AM

Which line is 17? The query line?

**Halbarad** · 09-15-2005, 11:50 AM

yeah it is

**0beron** · 09-15-2005, 02:31 PM

What's this doing in your query? :

PHP Code:


			
$affected_rows = $sql->id = $_SESSION[id];

**Halbarad** · 09-18-2005, 05:59 AM

i have allready removed that my query is now

PHP Code:


			
$query = mysql_query("UPDATE `members` 

SET `province` = '".$_POST['province']."',`kingdom` = '".$_POST['kingdom']."',`email` = '".$_POST['email']."',`msn` = '".$_POST['msn']."',`password` = '".$_POST['password']."' 

WHERE `id`='".$_SESSION['id']."'");