UPDATE causing me confusion

[baffled_in_UK]

Hi,

Can anybody give me a clue why this UPDATE is not updating.

there is a value of BD in the field code in the searchpost table.
the value of $key1 is BD

<?php

... connect stuff....

$sql="UPDATE searchpost SET gone=$gone, start_date=NOW(), end_date=$expdate, idx=$idx, login=$login, pass=$pass WHERE code=$key1";

... close stuff...

?>

Thanks

[jerdo]

Make sure you put single quotes around any text values you are trying to put into the DB. Example:
$sql="UPDATE searchpost SET gone='$gone', start_date=NOW(), end_date='$expdate', idx='$idx', login='$login', pass='$pass' WHERE code='$key1'";

[baffled_in_UK]

tried as you suggested..

$sql="UPDATE searchpost SET gone='$gone', start_date=NOW(), end_date='$expdate', idx='$idx', login='$login', pass='$pass' WHERE code='$key1'";

Still no update. Is the code ok ? Is there a better way to do it ?

Thanks

[erick_w]

Hmm... use mysql_query function! Try this:

<?

...... open dbconnections ......

$sql=mysql_query("UPDATE searchpost SET gone='$gone', start_date=NOW(), end_date='$expdate', idx='$idx', login='$login', pass='$pass' WHERE code='$key1'");

..... close dbconnections ......
?>

Hope it helps 😉

[The_Chancer]

In addition to just stating the query, echo it out when it is running so you can see what it is trying to put into the database, and put an or die statement in to let you know if it failed.

add:

$sql=mysql_query("UPDATE searchpost SET gone='$gone', start_date=NOW(), end_date='$expdate', idx='$idx', login='$login', pass='$pass' WHERE code='$key1'") or die (mysql_error());
echo "mysql_query(\"UPDATE searchpost SET gone='$gone', start_date=NOW(), end_date='$expdate', idx='$idx', login='$login', pass='$pass' WHERE code='$key1'\");"; 

Might give you more clues.

One other option is to use time() instead of NOW() as the date reference, if your column is a VARCHAR in the database - shouldn't be a problem - just change the query as below:

$sql=mysql_query("UPDATE searchpost SET gone='$gone', start_date='" . time() . "', end_date='$expdate', idx='$idx', login='$login', pass='$pass' WHERE code='$key1'") or die (mysql_error());

Just a thought...

[baffled_in_UK]

Thanks guys.

The mysql_query did the trick.