PHP Update statement

[AaronL]

Hello, I ran into a problem with my PHP code, my code works for adding a new record to the database, but for editing a record it doesn't work, and the MySql statement looks fine to me but the UPDATE doesn't seem to work on the record for some reason.

if($itemno == 0)
{
  $buildsql = "INSERT INTO products (title, catid, list, price, taxable, shipping, intshipping, calculator, description) ";
  $buildsql = $buildsql . "VALUES ('" . $title . "', '" . $catid . "', '" . $list . "', '" . $price . "', '" . $taxable . "', '" . $shipping . "', '" . $intshipping . "', '" . $calculator . "', '" . $description . "')";
  mysql_query($buildsql);
  $itemno = mysql_insert_id($con);
  echo "<table border='1' width='400' align='center'><tr><td bgcolor='#C0C0C0' align='center'><b>Item Added</b></td></tr>
<tr><td><b>Added item number: </b>" . $itemno . "<br><br><form action='imgupload.php' method='get'><b>
<input type='hidden' value=" . $itemno . " name='itemno' /><p align='right'><input type='submit' value='Upload Images'>
</p></td></tr>";
}
else
//Edit item
{
$buildsql = "UPDATE products SET title='" . $title . 
"' catid='" . $catid . "' list='" . $list . 
"' price='" . $price . "' taxable='" . $taxable . 
"' shipping='" . $shipping . "' intshipping='" . $intshipping . 
"' calculator='" . $calculator . "' description='" . $description . 
"' WHERE itemno='" . $itemno . "'";
  echo "<b>" . $buildsql . "</b>";
  mysql_query($buildsql);
}
 

What am I doing wrong? Thanks again for your help!

[prabirchoudhury]

ok.. you have a wrong Udate query expression .. should be "," after every field want to update..

 
this code should wrk to you 
 
$sql  = "UPDATE products SET title = '".$title."', catid = '".$catid."', list = '".$list."', ";
 $sql .=   "price = '".$price."', taxable = '".$taxable."', ";
 $sql .=   "shipping = '".$shipping."', ";
 $sql .=   "intshipping = '".$intshipping."', calculator = '".$calculator."', description = '".$description."'  ";            
 $sql .= "WHERE itemno = '".$itemno."' ";
 
  mysql_query($sql)or die("cant execute: $sql");
 
 

[AaronL]

You're a life saver, thanks!

[prabirchoudhury]

you are more then welcome ...