MySQL Update Help

[zevacronet]

I'm trying to update data in my MySQL database. Im getting an error, Query Empty, maybe someone can help me

require "functions.php";

$bday = $_POST['bday'];
$height = $_POST['height'];
$weight = $_POST['weight'];
$fcolor = $_POST['fcolor'];
$fanimal = $_POST['fanimal'];
$ton = $_POST['ton'];
$toff = $_POST['toff'];
$ame = $_POST['ame'];
$id = $_POST['id'];

$link = dbconnect();
$query = mysql_query("UPDATE 'ginfo' SET bday='$bday', 
height='$height', weight='$weight', fcolor='$fcolor', 
fanimal='$fanimal', ton='$ton', toff='$toff', ame='$ame' WHERE 
userid='$id'");
$result = mysql_query($query, $link)
or die('MySQL error: ' . mysql_error() . '<br>Query: ' . $query);
echo "Your bio has been updated...";

header ("Location: index.php");

[mogster]

I'm not sure, but try to remove the single quotes around the fieldname and use AND between every update, like
SET bday='$bday' AND height='$height'

instead of

SET bday='$bday', height='$height'

knutm

[cgraz]

first of all, you're using mysql_query on a string that was already executed by mysql_query.

$query = mysql_query("UPDATE 'ginfo' SET bday='$bday', 
height='$height', weight='$weight', fcolor='$fcolor', 
fanimal='$fanimal', ton='$ton', toff='$toff', ame='$ame' WHERE 
userid='$id'");
$result = mysql_query($query, $link)
or die('MySQL error: ' . mysql_error() . '<br>Query: ' . $query);

There's your code. I'd fix it so you don't it that way, and also print the query out to see what it looks like. Then you'll know if it's really empty or not. Try this:

$query = "UPDATE ginfo SET bday='$bday', 
height='$height', weight='$weight', fcolor='$fcolor', 
fanimal='$fanimal', ton='$ton', toff='$toff', ame='$ame' WHERE 
userid='$id'";
print "Query: " . $query . "<br>";
$result = mysql_query($query)or die(mysql_error());

I also dropped the single quotes around your table name. Your syntax is correct with using the commas. I don't know if AND will work on an UPDATE (it may, never tried it), but the way you are doing it now will work just fine assuming you don't mysql_query() twice and print out the string to make sure the variables actually have values.

Cgraz

[zevacronet]

duhhhhhhhh! What a stupid mistake. I didn't even notice the query thing. hahahha. Thanks guys!