Editting row problem - Dev Articles

Dev Articles Community Forums Sponsor:

July 11th, 2003, 01:58 AM

ignitionMX

Junior Member

Join Date: Apr 2003

Posts: 13

Time spent in forums: 5 m 3 sec
Reputation Power: 0

Editting row problem

This is my first time posting a PLEASE HELP!!! topic here... so... I really appreciate it if someone could help me.

The Problem

When I tell my script to update the information in the database, it somehow cannot update.

So, I'm wondering if there's a solution to my script.

PHP Code:


		
			
case 'editBuild':

        if ($row[permission] != 1) 

            {

                echo "<b>Error</b>";

                echo "<LI>You do not have the permission to view/edit the items on this page.<br>";

                echo "<b>Possible Solution(s)</b>";

                echo "<LI>Contact <a href=\"mailto:$admin_email\">$admin_name</a> to give you access rights.";

            }    

        else

            {        

                $database = mysql_connect("$dbHost","$dbUser","$dbPass");

                mysql_select_db("$dbDatabase", $database);

                

                    $id = $_POST["id"];

                    $user_name = $_POST["username"];

                    $email = $_POST["email"];

                    $birthday = $_POST["birthday"];

                    $biography = $_POST["biography"];

                    $permission = $_POST["permission"];

                    

                    $query  = "UPDATE users ";

                    $query .= "SET username = '$user_name', ";

                    $query .= "    email = '$email', ";

                    $query .= "    birthday = '$birthday', ";

                    $query .= "    biography = '$biography', ";

                    $query .= "    permission = '$permission', ";

                    $query .= "WHERE id = $id";



                $result = mysql_query($query);

                if ($result) { echo "The record has been editted."; }

                else { echo "Can't be done."; }

            }

    break;

Thanks in advance!

July 11th, 2003, 06:38 AM

digitallysmooth

you know how we do

Join Date: Jun 2002

Posts: 788

Time spent in forums: 2 h 34 m 21 sec
Reputation Power: 7

Take a look at this line: if ($row[permission] != 1)
I think you should have quotes around permission.

Also, why so much space between your query items

PHP Code:


		
			
"    email = '$email', ";

This doesn't need to have quotes

PHP Code:


		
			
 $database = mysql_connect("$dbHost","$dbUser","$dbPass");

mysql_select_db("$dbDatabase", $database);

single quotes would be faster

PHP Code:


		
			
 $id = $_POST["id"];

These may not solve your problem, but it will help. Also, you might want to echo out your query and post it here.

__________________
__________________________________________________ _
Wil Moore III, MCP | Integrations Specialist | Senior Consultant
Are You Listed...? | DigitallySmooth Inc.

July 11th, 2003, 08:11 AM

dhouston

Contributing User

Dev Articles Beginner (1000 - 1499 posts)

Join Date: May 2003

Location: Tennessee

Posts: 1,355

Time spent in forums: < 1 sec
Reputation Power: 7

I think your problem might be the comma between the last set clause and the WHERE. Try pulling that out and see if it works.

July 11th, 2003, 09:59 AM

_rainbow_

Registered User

Join Date: Jun 2003

Location: Lj, Slovenia

Posts: 27

Time spent in forums: 4 m 28 sec
Reputation Power: 0

Try using error reporting side of PHP. If you put after all of your SQL statements (mysql_connect, mysql_select_db, mysql_query) a

PHP Code:


		
			
or die("An error has occured: ".mysql_error()." : ".mysql_errno());

it will help you a lot. Another thing that I find strange is the way you put together your query. Try it this way:

PHP Code:


		
			
 $query**=*"UPDATE users SET*username*= '$user_name',*email*= '$email',*birthday*= '$birthday',*biography*= '$biography', permission*= '$permission',*WHERE*id*=*$id";

I appologize for the wide post [

]

July 11th, 2003, 10:56 AM

ignitionMX

Junior Member

Join Date: Apr 2003

Posts: 13

Time spent in forums: 5 m 3 sec
Reputation Power: 0

Thanks dhouston! Yes, it was the comma.

To laidbak - I'll keep those single quotes in mind =D (I'm still a bit rusty on PHP.)

and to rainbow - I didn't like long strings or whatever you guys call it, so I made it the way I made it.

Thanks for the error thing.

Thread Tools	Search this Thread
Email this Page	Search this Thread: Advanced Search
Display Modes	Rate This Thread
Linear Mode Switch to Hybrid Mode Switch to Threaded Mode	Rate This Thread: