Question

I have a strange mysql-thing going on here, it is about the following code:

$res = mysql_query("SELECT * FROM users WHERE group='".$group."'");
if (mysql_num_rows($res)==1) {

$row = mysql_fetch_assoc($res); 
$uid = $row['uid'];

$user_update = mysql_query("UPDATE fe_users SET group = 5 WHERE  group='".$group."'");           

return 'ok';

} else {

return 'not ok';

}   

I am checking, if there is a user with the group = $group. If so, the group is updated to 5 and after that the string “ok” is returned, if no user with group=$group exists, as you can see the string “not ok” is returned.

This should be very easy, but the problem now is, that if there is a user with group=$group, the update is done correctly, but instead of returning “ok”, php returns “not ok”, as if the change from the update is been taken into account for the above executed select retroactively. I dont understand this. Any help would be really appreciated.

Thanx in advance,
Jayden

Answer 1

I think ‘group’ is a reserved keyword that you have used as a field name, change it or use like

$res = mysql_query("SELECT * FROM users WHERE `group`='".$group."'");

and

$user_update = mysql_query("UPDATE fe_users SET `group` = 5 WHERE  `group`='".$group."'");

and you can use count($res)==1 instead of mysql_num_rows($res)==1 if it is a problem.

Reference: Mysql Reserved keywords.