[SOLVED] problem deleting

[Jessica]

To turn on error reporting:

ini_set('display_errors', 1);
error_reporting(E_ALL);

[amylou]

i did that and the error is that i have undefined variables

 

last=$last and first=$first.

 

how would I define the variables in the delete code

 

thanks for your help

[Jessica]

Where are they coming from?

last=$last doesn't make any sense. Are you saying it says undefined variable, or undefined index? Post the ACTUAL error and the lines of code it refers to.

[amylou]

that is the error i am getting

 

 

Notice: Undefined variable: last in C:\apache2triad\htdocs\index.php on line 14 Notice: Undefined variable: first in C:\apache2triad\htdocs\index.php on line 14

 

 

this is the line of code

$query ="DELETE FROM timeoff WHERE last = '$last' AND first = '$first' ";

 

 

[Jessica]

So where is $last defined? You're obviously not setting it, so where is it supposed to come from? Is it in the URL (use $_GET['last']). Is it posted? Use $_POST. Session? use $_SESSION.

Otherwise, it's made up and has no use

[amylou]

the program lets a person go to a form input first and last name click submit and the results are shown with their name , date, adn a check box

 

after the last checkbox there is a link for delteing

 

not sure how i would validate them in this code.

 

 

if($_GET['m'] == 'del' )
{
$requestId = $_GET['id'];
$query ="DELETE FROM timeoff WHERE last = '$last' AND first = '$first' ";

$result= mysql_query($query) or die(mysql_error());

[Jessica]

So $last needs to be defined by:

$last = $_POST['last'];

Also, you need to validate it. If someone puts in "O'Hara" it could break your code. It's open to SQL injection and a bunch of other stuff.

[amylou]

Notice: Undefined index: last in C:\apache2triad\htdocs\index.php on line 13 Notice: Undefined index: first in C:\apache2triad\htdocs\index.php on line 14

 

 

the line of code that it goes to is the

 

$last=$_POST['last'];

$first=$_POST['first']

 

maybe i did not put it in the right spot i put it before the if statement

[Jessica]

It sounds like nothing is being posted for those fields.