Update Query Problem

rakennedy75

I am using an update query to change information in a database. Does anyone see anything wrong wit this code. Its not erroring out, just won't work update the data.

$connection = db_connect();
$query = "UPDATE image_table SET


				   imageID = '$imageID',
                   imagePath = '$imagePath', 
                   category = '$category', 
                   jobTitle = '$jobTitle',
                   startDate = '$startDate',
                   finishDate = '$finishDate',
                   jobDetails = '$jobDetails',
                   actStat = '$actStat',
				   WHERE imageID = '$imageID'";

   $result = mysql_query($query, $connection);
   if ($result)
   {
    	echo "your item has been changed";
   }  else {
   		echo "could not make your change";
   }

Thanks for your time

kevinsequeira

take a look at this

http://ca.php.net/manual/en/function.mysql-affected-rows.php

reg
kevin

ScubaKing22

You could also try:

if (mysql_error)
{
echo mysql_error;
}

superwormy

Change this:

$result = mysql_query($query, $connection);

To this:

$result = mysql_query($query, $connection) or die (mysql_error());

Also, be careful, mysql_query() will return TRUE if the query executes correctly, it indicates NOTHING AT ALL about whether or not a record was actually changed.

rakennedy75

First off, thanks for the response to my post.

I put the error check in and I am getting this error. Never seen this one before:

Duplicate entry '0' for key 1

Any ideas how to prevent this?

superwormy

You usually get that error when you have a duplicate PRIMARY KEY or a duplicate UNIQUE field.

Solution: Don't try and update something and give it a key thats already in use...

Generally speaking, once a key has been assigned, you should never be updating or changing that key, you'll break references to it in other places, and end up with problems like this. So in your example, I assume imageID is the PRIMARY KEY, so stop updating it. Just update the other fields.

rakennedy75

Thanks a lot, that was the problem, works perfectly now.

Thanks for taking the time.