Originally Posted by grumpy

add
echo $sql;

and see if the line is correct or post it here

Originally Posted by Jakez

When something isn't working and I can't for the life of me figure out why I usually just start "dieing" stuff out, try:
die($sql)

Then you can see exactly what it's telling mysql and copy/paste it into phpmyadmins SQL query box. Haven't used mysql_error() in a while but phpmyadmin will give you some good info about the query if it fails or not.