have to double click link...something wrong in 'if'

[DKY]

Can anyone help me figure out why I have to double click my 'Delete' link? the first time I click it, it puts the link in the address bar, but it doesnt delete. the second time it deletes. Its as if it doesnt go through the if statement until it is clicked the second time. What's wrong with my if statement?

<? 
		while ($row = mysql_fetch_assoc($result)) { 
            $delete = '<a href="?mode=delete&main_id='.$row['unique_identifier'].'">Delete</a>'; 
            echo "<table cellpadding='3' cellspacing='0' width='100%' border='0' 
bordercolor='#000000'><tr><td style=\"border-top-style: solid; border-top-width: 1\" 
id='questions'>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;-&nbsp;&nbsp;{$row['questions']}</td>
<td width='50' valign=bottom style=\"border-top-style: solid; border-top-width: 1\">
$delete</td></tr>\n"; 
        } 

echo "</table>\n"; 

if (((isset($_GET["mode"])) && ($_GET["mode"] == "delete")))
 { 
        $deletequery = "DELETE FROM `$table` WHERE unique_identifier='{$_GET['main_id']}'"; 
        $deleteresult = mysql_query($deletequery) or die(mysql_error().'<br />'); 
 }

 ?>

[devinemke]

look at the order that you are doing things in your script. you are:

1. displaying the data
2. deleting the data

when you click once you will still see the data since it has not yet been deleted.

[DKY]

so, how do i get it to 'refresh' the display part?

[devinemke]

Originally posted by DKY
so, how do i get it to 'refresh' the display part?

you don't. you reverse the order. do the delete first, then pull and display the data.

[DKY]

I changed it to this and still have to click it twice. Here's my page if you want to test it out. http://dky.sytes.net/kq/

<? 

	if ((isset($_GET["mode"])) && ($_GET["mode"] == "delete"))
 { 
        $deletequery = "DELETE FROM `$table` WHERE unique_identifier='{$_GET['main_id']}'"; 
        $deleteresult = mysql_query($deletequery) or die(mysql_error().'<br />'); 
 }
while ($row = mysql_fetch_assoc($result)) { 
            $delete = '<a href="?mode=delete&main_id='.$row['unique_identifier'].'">Delete</a>'; 
            echo "<table cellpadding='3' cellspacing='0' width='100%' border='0' bordercolor='#000000'>
<tr><td style=\"border-top-style: solid; border-top-width: 1\" id='questions'>
-&nbsp;&nbsp;{$row['questions']}</td><td width='50' valign=bottom 
style=\"border-top-style: solid; border-top-width: 1\">$delete</td></tr>\n"; 
        } 

echo "</table>\n"; 
 ?>

I think something's wrong with this part.

if ((isset($_GET["mode"])) && ($_GET["mode"] == "delete"))

[devinemke]

you need to do the following things in the following order:

1. connect to the database
2. execute your DELETE query (if the proper conditions are true)
3. execute your SELECT query
4. echo your HTML ouput (based on the results of the SELECT query)

[DKY]

ahaa, got it. Needed to put it before the select query, thanks devinemke. Is that the proper order for all php pages?