MySQL Update table wierdness

[cricher]

I'm using this:

$sql = "UPDATE form_subcat " .
               "SET  `id`=" . $_POST['id']. "' " .
               " WHERE `name`='" . $_POST['name'] .
               "', `desc`='" . $_POST['desc'] .
               "', `cat_id`='" . $_POST['cat_id'];

to edit a sql entry where the Admin can change pretty much everythng but the ID, however when I hit submit it creates a new table entry with a different ID also it doesn't delete the old entry 😕
Can anyone help please?

[cgraz]

are desc and cat_id part of your WHERE clause? If so, you need AND separating them. If not, move them above the WHERE clause.

[cricher]

um no they're not, i appear to have cocked this up lol. Basically i need it to update all the fields, however, the ID needs to stay the same. The table entry should be defined by the ID.

[laserlight]

Looks like you got it mixed up. You're probably looking for something like:

$sql = "UPDATE form_subcat " .
               "SET `name`='" . $_POST['name'],
               "', `desc`='" . $_POST['desc'] .
               "', `cat_id`='" . $_POST['cat_id'] .
               " WHERE `id`=" . $_POST['id'];

Of course, you should escape the incoming variables appropriately or risk SQL injection.