MySQL Edit records using PHP

Question

I am not able to edit the field and update it. The name field displays the text":

Notice: Undefined variable: row in D:\XAMPP\htdocs\test\test5.php on line 31

instead of the value. Might have done some silly code error, I'm a naivete.

<!DOCTYPE html>
<html>
<body>
<?php
define('DB_NAME', 'test');
define('DB_USER', '');
define("DB_PASSWORD", '');
define('DB_HOST', 'localhost');

$link = mysql_connect(DB_HOST, DB_USER, DB_PASSWORD);
$db_selected = mysql_select_db(DB_NAME, $link);

if(isset($_GET['del']))
{
    $value1=$_GET['del']; 
    $rs = mysql_query("SELECT * from `add` where empid ='".$value1."'"); 
    $row = mysql_fetch_array($rs);
}
?>
    <form action="test7.php" method="post">
        <ul>
            <li>
                Name:</br>
                <input type="text" name="name" value="<?php echo $row[0]; ?>">
            </li>
            <li>
                <input type="submit" value="UPDATE">
            </li>
        </ul>
    </form>
</body>
</html>

Your $row variable is not set if you don't have a del GET parameter. That is why you get an error when you try to read that variable on line 31 with <?php echo $row[0]; ?> — AeroX
– AeroX, Commented Dec 23, 2014 at 15:55

vaso123 · Accepted Answer · 2014-12-23 15:56:25Z

3

You can check, is the $row[0]is exists. If there are no del parameter in the $_GET then $row will not exists, becuase you are creating $row in the condition.

<input type="text" name="name" value="<?php echo (isset($row[0]) ? $row[0] : ''); ?>">

NOTE

Do not use mysql functions. They are deprecated. Use mysqli or PDO instead.
Avoid sql injection by escaping your variables from outside, or use prepared statements.

answered Dec 23, 2014 at 15:56

vaso123

12.4k4 gold badges38 silver badges68 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

user645 Over a year ago

Now getting a blank field

user645 Over a year ago

How can i chck wether $row[0] exists?

vaso123 Over a year ago

As you can see in my code, I am calling the isset() function. That syntax, what I've used, a shorter version of if (isset($row[0])) { echo $row[0]; } else { echo ''; }

Kevin · Accepted Answer · 2014-12-23 15:55:21Z

1

You are echo'ing $row[0] in your form without declaring it. You need to initialize and declare $row outside the if statement and before you echo it.

answered Dec 23, 2014 at 15:55

Kevin

8623 gold badges16 silver badges35 bronze badges

2 Comments

user645 Over a year ago

How can i chck wether $row[0] exists?

Kevin Over a year ago

@user645 <?php echo (isset($row[0]) ? $row[0] : ''); ?>

geekido · Accepted Answer · 2014-12-23 16:00:31Z

0

updated code.

<?php

$row = "";
 if(isset($_GET['del']))
    {
     $value1=$_GET['del']; 
     $rs = mysql_query("SELECT * from `add` where empid ='".$value1."'"); 
     $row = mysql_fetch_array($rs);
    }
?>

<form action="test7.php" method="get">
    <ul>
        <li>
            Name:</br>
            <input type="text" name="name" value="<?php echo $row[0]; ?>">
        </li>

        <li>
            <input type="submit" value="UPDATE">
        </li>
    </ul>
</form>

you must declare $row first. if you declare it in if, you can't use it outside. Also change your form method to GET

edited Dec 23, 2014 at 16:00

answered Dec 23, 2014 at 15:54

geekido

1715 bronze badges

1 Comment

geekido Over a year ago

I've copied your code and change two sections. Would you please share your code using pastebin or something

Collectives™ on Stack Overflow

MySQL Edit records using PHP

3 Answers 3

3 Comments

2 Comments

1 Comment

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

2 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Linked

Related