How to solve Notice: Undefined index: id in C:\xampp\htdocs\invmgt\manufactured_goods\change.php on line 21 [duplicate]

Question

I have a problem with my PHP code saying that "Notice: Undefined index" I am sure its very simple, since I am a beginner i am not getting well what is wrong exactly so please help me.

Here's the code

<?php require_once('../Connections/itemconn.php'); ?> <?php $id=$_GET['id']; $query=mysql_query("select * from manuf where id='$id' ")or die(mysql_error()); $row=mysql_fetch_array($query); ?> <form action="updateprice.php" method="post" enctype="multipart/form-data"> <table align="center"> <tr> <td> <label><strong>Item Name</strong></label></td> <td> <label> <?php echo $row['itemname']; ?></label><input type="hidden" name="id" value="<?php echo $id; ?> " /> <br /></td> </tr> <tr> <td><label><strong>Unit price </strong></label></td> <td> <input type="text" name="pass" value="<?php echo $row['unitprice']; ?> " /><br /></td> </tr> <tr> <td> <input type="reset" name="Reset" value="CANCEL" /> <br></td> <td> <input type="submit" name="Submit2" value="Update" /> </td> </tr> </table> </form> </body> </html>

You are using $_GET['id'] but there is no ID in your URL. Are you sure you're not mixing up $_POST and $_GET ? — Matheno
– Matheno, Commented Oct 31, 2014 at 8:42

Pupil · Accepted Answer · 2017-01-16 03:49:56Z

You are not getting value of $id=$_GET['id'];

And you are using it (before it gets initialised).

Use php's in built isset() function to check whether the variable is defied or not.

So, please update the line to:

$id = isset($_GET['id']) ? $_GET['id'] : '';

for me removing form action did the thing, its changed the link before query ran so i removed the action from form.

Rakesh Sharma · Accepted Answer · 2014-10-31 08:52:16Z

if you are getting id from url try

$id = (isset($_GET['id']) ? $_GET['id'] : '');

if getting from form you need to use POST method cause your form has method="post"

 $id = (isset($_POST['id']) ? $_POST['id'] : '');

For php notices use isset() or empty() to check values exist or not or initialize variable first with blank or a value

$id= '';

No, remove the empty from your answer (see link at bottom of my answer as to why you shouldn't use empty)

ajitpawarink · Accepted Answer · 2014-10-31 08:48:52Z

3

Simply add this

$id = ''; if( isset( $_GET['id'])) { $id = $_GET['id']; }

answered Oct 31, 2014 at 8:48

ajitpawarink

4446 silver badges14 bronze badges

Collectives™ on Stack Overflow

How to solve Notice: Undefined index: id in C:\xampp\htdocs\invmgt\manufactured_goods\change.php on line 21 [duplicate]

3 Answers 3

1 Comment

2 Comments

Comments

Linked

Hot Network Questions