sql error

[PaulaRun]

Does anybody know why I'm getting the following error:

Could not successfully run query from DB: You have an error in your SQL syntax near 'where prod_no = 1' at line 1

This is my query:

$query = "update products set prod_name = '".$_POST['name1']."', price1 = ".$_POST['price1'].", where prod_no = ".$_POST['prodno']."" or die(mysql_error());

The table:

while($row = mysql_fetch_assoc($result)) {
   	echo "<br><font size=\"2\">";
	echo mysql_affected_rows()." record found.";
	echo '<form action="' . $_SERVER['PHP_SELF'] . '" method="post" onSubmit="return Validator(this)">';
	echo '<p><font size="2">';
	echo "<input type=\"hidden\" name=\"prodno\" value=\"".$row["prod_no"]."\">";	
	echo "<input type=\"text\" name=\"name1\" size=\"20\" maxsize=\"30\" value=\"".$row["prod_name"]."\"><font size=\"2\">&nbsp; Product Name&nbsp; </font><br>";
	echo "<input type=\"text\" name=\"price1\" size=\"5\" maxsize=\"5\" value=\"".$row["price1"]."\"><font size=\"2\">&nbsp; Price 1<br></font>";
</form>

Any suggestions would be greatly appreciated!!

[ale]

Hi,
i'm principiant of php-mysql.
I think that error is near WHERE:

.......    ->  ,  <-     where prod_no = ".$_POST['prodno']."" or die(mysql_error());

bye

[mogster]

Ummm...
Yes, if this is your query? :

$query = "update products set prod_name = '".$_POST['name1']."', price1 = ".$_POST['price1'].", where prod_no = ".$_POST['prodno']."" or die(mysql_error());

You can't have the mysql error specified behind the construction of the query - you must compile, then do the query with the error-safing:

$query = "update products set prod_name = '".$_POST['name1']."', price1 = ".$_POST['price1'].", where prod_no = ".$_POST['prodno']."" 
$do_the_query = mysql_query($query) or die(mysql_error());

knutm

[PaulaRun]

Ale - THANKS!! That was it, a stupid comma before the where clause! Truns out the whole statement was much larger but I shortened it to test and left the comma in by accident. (my bad)

Mogster - I was able to run the query with the mysql error behind it. (?)

Anyway, thank you both for your help!

[mogster]

Ah, well - I'm wrong again sigh :rolleyes: 😃

knutm