Problem mit Datensatz löschen aus Tabelle

[chucky]

Hallo ihr lieben. ich habe das problem, das wenn ich mein script ausführe ich zwar angezeigt bekomme das der datensatz gelöscht sei, der datensatz aber noch da ist. bitte helft mich

<?
include "mysql.php";
$id = $HTTP_GET_VARS['ID'];
$delete  = $HTTP_GET_VARS['delete'];

if($delete){
    mysql_query($sql) OR die("<pre>\n".$sql."</pre>\n".mysql_error());
    $sql = "DELETE FROM `navigation` WHERE `ID` = ".$HTTP_GET_VARS['ID']."";
    echo '<script type="text/javascript">alert("Erfolgreich gelöscht");hystory.back();</script>';
}
?>

entschuldigt mal wieder meine formlosigkeit die kommt auch noch wenn alle fehler raus sind.

[Blackbird]

Schreibe die Zeile die mit $sql beginnt vor die Zeile die mit mysql_query beginnt. Ich denke dann sollte es funktionieren.

[Falk Prüfer]

Hallo,

einige Anmerkungen:
HTTP_GET_VARS ist seit PHP 4.0.1! veraltet und sollte nicht mehr verwendet werden. Stattdessen gibt es die Superglobale $_GET!
Eingabewerte von Aussen sollten NIE! ungeprüft und unmaskiert in SQL-Strings verkettet werden! Siehe SQL-Injection.
Überleg dir mal, wie dein SQL-String aussehen würde, wenn einfach jemand dein Script mit dem Parameter ?ID=ID&delete=1 aufruft.

Gruß Falk