What's wrong with this syntax

[NZ_Kiwis]

$result = mysql_query ("UPDATE members SET email = '" . $_POST['Email1'] . "', password = '" . $_POST['NewPassword'] . "', districts = '" . $_POST['districts'] . "' WHERE id = " . $_GET['ID'] . ");

[NogDog]

Get rid of the ." right before the closing parenthesis.

[kdhayanandan]

You missed the " after closing paranthesis..

$result = mysql_query ("UPDATE members SET email = '" . $POST['Email1'] . "', password = '" . $POST['NewPassword'] . "', districts = '" . $POST['districts'] . "' WHERE id = " . $GET['ID'] . ")";

NZ_Kiwis;10896483 wrote:

$result = mysql_query ("UPDATE members SET email = '" . $_POST['Email1'] . "', password = '" . $_POST['NewPassword'] . "', districts = '" . $_POST['districts'] . "' WHERE id = " . $_GET['ID'] . ");

[NogDog]

kdhayanandan;10896491 wrote:

You missed the " after closing paranthesis..

$result = mysql_query ("UPDATE members SET email = '" . $POST['Email1'] . "', password = '" . $POST['NewPassword'] . "', districts = '" . $POST['districts'] . "' WHERE id = " . $GET['ID'] . ")";

That won't parse, either. The ")" is for closing the mysql_query() function; it's not part of the query.

[bradgrafelman]

Also, get rid of the possible SQL injection vulnerability by escaping the user-supplied data (e.g. by using [man]mysql_real_escape_string/man). :p