Delete from database

[adammc]

Hi Folks,
Im having probs with the code below. It wont delete the entry from the database. The page just refreshes?

Can anyone help?

<br><br><br>
<center><font color='navy' face='verdana' size='2'><b>Your current Job Alerts are:</b></font></center><br><br>

<?

# setup SQL statement
$SQL = " SELECT * FROM jobalert ";
$SQL = $SQL . " WHERE uname = '$uname' ";

# execute SQL statement
$retid = mysql_db_query($db, $SQL, $cid);

# check for errors
if (!$retid) { echo( mysql_error()); }
else {

    # display results
    while ($row = mysql_fetch_array($retid)) {
        $uname = $row["uname"];
        $jobcategory = $row["jobcategory"];
    $email = $row["email"];


        echo ("<table width=90% align='center'><tr><td><font face='verdana' size='2'><b>Job Category:</b>&nbsp;&nbsp; $jobcategory</font></td><td align=right><a href='job-alerts.php?jobcategory=$jobcategory&uname=$uname&task=del'><font face='verdana' size='2'>Delete</td></tr></table><br>");
    }

}

 # delete from database
if ($task=="del") {

    $SQL = " DELETE FROM jobalert ";
    $SQL = $SQL . " WHERE jobcategory = $jobcategory, uname = $uname ";
    mysql_db_query($db, $SQL, $cid);

}
?>

[BIOSTALL]

setup SQL statement

$SQL = " SELECT * FROM jobalert ";
$SQL = $SQL . " WHERE uname = '$uname' ";

this will produce a double space between jobalert and WHERE. Not sure if that has nefin to do with it. Why not just do:

$SQL = "SELECT * FROM jobalert WHERE uname='$uname'"??

# execute SQL statement
$retid = mysql_db_query($db, $SQL, $cid);

this should be:

$retid = mysql_db_query($SQL, $cid);

Hope that helps.

BIOSTALL

[adammc]

Thanks for the tip, got it all sorted 🙂