[RESOLVED] Instead of updating values they are removed. Why?

god0fgod

Hello. I'm having a problem while updating a row in a database. Instead of updating it with variables (They aren't empty since I've tested them) the query I made removes any value in the fields I want to change.

$theme = mysql_escape_string($_POST['theme']);
switch ($theme){
case "default":
$extras = $_POST['theme_extras1'];
break;
case "classic":
$extras = $_POST['theme_extras2'];
break;
case "custom":
$extras = $_POST['theme_extras3'];
break;
}
$extras = mysql_escape_string(implode("}{",$extras));
echo $extras . " - " . $theme; //variables seem fine.
$sql = "UPDATE users SET theme = '$theme', theme_extras = '$extras' WHERE username='$username'"; 
$result = mysql_query($sql); //Problem with query.
if(!$result){
echo "<span style='color:red;'>" . mysql_error() . "</span>"; //no error
}else{
echo "<span style='color:green;'>The theme has been altered.</span>";
}

Anyone know what could be the problem here?

Thanks for any reply.

leatherback

Have you echoed the actual query? What does that give you? I am not sure, but I think that single quotes hamper the replacement of variables by their values. For easy of reading I would normally use '".$variable."' in queries.

Also, as this seems to be a userprofile you are updatng: Set your limit to 1, so you will not affect more than 1 record.

god0fgod

The query returns with 1 which is as expected. ' is no different to '" . as expected. There is no need for a limit. My website is designed to allow only one of each username.

Thanks for your reply leatherback. It still doesn't work however.

I doubt it is anything to do with the PHP code. could anything be wrong with the table?

It isn't resolved actually. I'm getting the same problem again. It only seemed a one off.

NogDog

Have you echoed out $sql after you define it to make sure it's what you expect it to be?

god0fgod

Sorry leatherback. I think that's what you meant me to do. Anyway, I see no problem in this:

UPDATE users SET theme = 'default', theme_extras = 'High(70%)}{Default' WHERE username='god0fgod'

god0fgod

Well, I've just got this from my webhost:

The database connection string that you were using the file account.php was wrong and for this reason it was not connecting to the server. Instead of connecting to the MySQL server, it was directly trying to access the database. I have added the connection string and now all set fine.

however it still doesn't work. Do you know what this means? I may have accidentally over ridden any changes made between the time of the fix and me finding out.