kleiner adminbereich für gästebuch

[cille]

mhhh, also bei mir funktioniert es, habe dein code genommen, nur die b enutzedaten anders eingestellt.

 

[Wambo]

mhhh, also bei mir funktioniert es, habe dein code genommen, nur die b enutzedaten anders eingestellt.

Komisch,
woran kann das denn liegen das das Script bei mir nicht funktioniert, weil nur an den benitzerdaten kann es ja nicht liegen.

 

[Dennis Wronka]

Also wie ich das seh macht der Code in dieser Form nicht gerade viel Sinn:

$username = "admin";            // Dein username
$password = "admin";     // Dein Password

if (isset($_GET["admin"]) & $_GET["admin"] == $username || isset($_GET["$password"])) {
    echo "<h3>Administrierbereich</h3>";
    $sql = "SELECT * FROM $dbtabelle";
    $result = mysql_query($sql);
    while ($row=mysql_fetch_assoc($result)) {
        echo "<a href=$_SERVER[PHP_SELF] .$password. = $row[id]'>Eintrag löschen</a>";
        foreach ($row as $name => $key) {
            echo $key;
        }
        echo "<br><br>";
    }
}

if (isset($_GET[$password]))
 {
    $sql =  'DELETE 
            
            FROM 
            
            '.$dbtabelle.' 
            
            WHERE
            
            id='.mysql_real_escape_string($_GET[$password]);

}

Sowohl $_GET[$password] als auch $_GET["$password"] entsprechen $_GET['admin'], beim ersten IF (wo uebrigens ein & fehlt, denn es der logisches UND wird durch && repraesentiert und nicht durch &) wird also ueberprueft ob $_GET['admin'] gesetzt ist und $_GET['admin'] gleich $username (also admin) ist oder $_GET['admin'] gesetzt ist.
Hoert sich verwirrend an? Ist es auch.
Das duerfte dann dafuer sorgen, dass, solang $_GET['admin'] gesetzt ist der darauf folgende Abschnitt (also das SELECT und dessen Ausgabe), durchgefuehrt werden.
Beim 2. IF wird dann wieder lediglich gecheckt ob $_GET['admin'] gesetzt ist, denn wie wir bereits oben festgestellt haben entspricht ja, da $password='admin' ist, $_GET[$password] $_GET['admin']. Wenn dies der Fall ist wird der Eintrag mit der ID $_GET[$password] (also $_GET['admin']) geloescht.
$_GET['admin'] kann also einen Usernamen beinhalten, oder aber eine (wahrscheinlich) numerische ID.
Na, verwirrt? Ich auch.
Wenn nun oben die Bedingung $_GET['admin']==$username zutrifft wird im 2. Block also versucht den Wert mit der ID 'admin' zu loeschen.

Also wenn der Code mal nicht voll daneben ist, dann weiss ich auch nicht...

 

[Wambo]

Also wie ich das seh macht der Code in dieser Form nicht gerade viel Sinn:
Sowohl $_GET[$password] als auch $_GET["$password"] entsprechen $_GET['admin'], beim ersten IF (wo uebrigens ein & fehlt, denn es der logisches UND wird durch && repraesentiert und nicht durch &) wird also ueberprueft ob $_GET['admin'] gesetzt ist und $_GET['admin'] gleich $username (also admin) ist oder $_GET['admin'] gesetzt ist.
Hoert sich verwirrend an? Ist es auch.
Das duerfte dann dafuer sorgen, dass, solang $_GET['admin'] gesetzt ist der darauf folgende Abschnitt (also das SELECT und dessen Ausgabe), durchgefuehrt werden.
Beim 2. IF wird dann wieder lediglich gecheckt ob $_GET['admin'] gesetzt ist, denn wie wir bereits oben festgestellt haben entspricht ja, da $password='admin' ist, $_GET[$password] $_GET['admin']. Wenn dies der Fall ist wird der Eintrag mit der ID $_GET[$password] (also $_GET['admin']) geloescht.
$_GET['admin'] kann also einen Usernamen beinhalten, oder aber eine (wahrscheinlich) numerische ID.
Na, verwirrt? Ich auch.
Wenn nun oben die Bedingung $_GET['admin']==$username zutrifft wird im 2. Block also versucht den Wert mit der ID 'admin' zu loeschen.

Also wenn der Code mal nicht voll daneben ist, dann weiss ich auch nicht...

Wie soll ICH das verstehen, wenn du selber das nicht mal verstehst?

 

[Dennis Wronka]

Es reicht fuer Dich erstmal zu wissen, dass der Code Kaese ist und am besten neu geschrieben werden sollte.