Coding Problems

[Kirk Y]

I've got messages being pulled from a database and I'm trying to create an option for the other administrators to remove the messages by clicking an image. I'm using the following code:

PHP Code:

$blast_msg = $db->query_read(" 
    SELECT * 
    FROM blastmsg
");
while ($blast = $db->fetch_array($blast_msg)) 
    { 
        $user = $blast['user']; 
        $date = $blast['date']; 
        $message = $blast['message']; 
        $id = $blast['blastid'];
    }
if ($_GET['do'] == 'delete')
{
 if (!$blast = $db->query_first("SELECT blastid FROM blastmsg WHERE blastid = " . intval($_GET['id'])))
 {
  eval(standard_error(fetch_error('non_existant')));
 }
 
 $db->query("DELETE FROM blastmsg WHERE blastid = '$id'");
 eval(print_standard_redirect('blast_del'));  
} 


The problem is that upon clicking the image, the wrong message is being erased.

I'm using this as the URL:

PHP Code:

<a href='blast.php?{$vbulletin->session->vars['sessionurl']}do=delete&amp;id=$id'> 


Thanks in advance for any ideas.

[calorie]

Your delete query is using the last ID set from the while loop instead of something like the following:

$db->query("DELETE FROM blastmsg WHERE blastid = " . intval($blast['blastid']));

[Kirk Y]

D'oh! Can't believe I missed that -- thanks Calorie.