Chp 12: update_user.php doesn't works

DMatt · September 7th, 2010, 05:29 PM

Hi

I've a problem with the original update_user.php file download here..

The file must be update the informations of a "single" user, set by the user_id..
Instead when i press the "update" button ALL the user are updated!!!

Now i've my 3 users with the same informations! same first name, last name, email, city, state and hobbies! This is a very big problem..

How can i fix the problem??

DMatt · September 8th, 2010, 04:23 PM

Ok Ok i solve it! More simple than i thought!

There is the original portion of code that update the user info

PHP Code:


			
$query = 'UPDATE site_user u, site_user_info SET

            username = "' . mysql_real_escape_string($username, $db) . '",

            first_name = "' . mysql_real_escape_string($first_name, $db) . '",

            last_name = "' . mysql_real_escape_string($last_name, $db) . '",

            email = "' . mysql_real_escape_string($email, $db) . '",

            city = "' . mysql_real_escape_string($city, $db) . '",

            state = "' . mysql_real_escape_string($state, $db) . '",

            hobbies = "' . mysql_real_escape_string(join(', ', $hobbies), $db) . '"

          WHERE

            u.user_id = ' . $user_id;

and there is the modified code, now it works

PHP Code:


			
$query = 'UPDATE site_user u, site_user_info i SET

            username = "' . mysql_real_escape_string($username, $db) . '",

            first_name = "' . mysql_real_escape_string($first_name, $db) . '",

            last_name = "' . mysql_real_escape_string($last_name, $db) . '",

            email = "' . mysql_real_escape_string($email, $db) . '",

            city = "' . mysql_real_escape_string($city, $db) . '",

            state = "' . mysql_real_escape_string($state, $db) . '",

            hobbies = "' . mysql_real_escape_string(join(', ', $hobbies), $db) . '"

          WHERE

            u.user_id = ' . $user_id . ' AND i.user_id = ' . $user_id;

very very simple

bopjo1 · October 9th, 2010, 03:57 PM

Thanks for solving this. I just ran accross the same error today. Everytime I updated a user it updated every other user with the same info.

bopjo1 · October 9th, 2010, 04:04 PM

..but I am getting this errors:

Warning: mysql_num_rows() expects parameter 1 to be resource, null given in C:\xampp\htdocs\Tutorials\wrox\update_user.php on line 57

Warning: mysql_free_result() expects parameter 1 to be resource, null given in C:\xampp\htdocs\Tutorials\wrox\update_user.php on line 61

bopjo1 · October 9th, 2010, 04:12 PM

oops...just solved those errors. I forgot the code to submit the query.

DMatt · October 17th, 2010, 04:30 PM

to many errors in this book...

bopjo1 · October 17th, 2010, 06:06 PM

DMatt - did you work through the later chapters? The online store and the bulletin board? I'm wondering it if it even worth the trouble.

DMatt · October 18th, 2010, 05:36 PM

I've ended the book, the 2 final chapters have some errors.. i don't remember exactly wich are.. however i suggest you finish the book, is interesting