DELETE wird nicht ausgeführt

[theaims]

Hallo

Also ich habe eine dynamische dropdown box. Das funtzt auch alles. Nur der delet befehl wird nicht ausgeführt der danach folgt. Ich habe mal ein wenig das ganze komentiert. Ich hoffe iher könnt mir helfen.

<form action="<?php echo $_SERVER['PHP_SELF'];?>" method="post"> 
<?php
$sql = "SELECT id FROM $name";
    $ergebnis = mysql_query($sql) or die(mysql_error());    
       if($ergebnis)
     {
       echo "<select name='id' size='1' onchange='document.forms.form1.an_id.value=this.form.id.options[this.form.id.selectedIndex].value'>";  
        
        echo "<option value='' selected></option>";
           while($row=mysql_fetch_array($ergebnis))
               {
               
               echo "<option value="."'".$row['id']."'".">".$row['id']."</option><br>";
                }
      echo "</select>";
      echo '<input type="submit" name="submit" value="Löschen">';

// bis hier hin geht alles

// danach wird mir noch nicht mal die sql ausgegeben.
                     }
 if(isset($row['id'])) {
   {
        } 
$sql = "DELETE FROM `$name` WHERE '$id'";
echo $sql;
$query = mysql_query($sql) or die(mysql_error()); 
}

[wahsaga]

Re: DELETE wird nicht ausgeführt

Und was ergaben deine Kontrollausgaben?

[Benny-one]

$sql = "DELETE FROM `$name` WHERE '$id'";


Where $id? ja, du musst schon noch die Spalte dazu schreiben...

und du meinst auch sicher $id und nicht $row['id']?

btw:

if(isset($row['id'])) {
   {
        } 

da ist eine { zu viel

[theaims]

Also bei WHERE wollte ich bewirken das die spalte in der die ausgewählte id, die oben im formular feld ausgewählt ist komplett gelöscht ist. Wie mach ich das?

[Benny-one]

sofern die Spalte wirklich id heißt

WHERE id='$id'

[theaims]

Habe das jetzt so gemacht. Aber ich erhalte kein SQL echo.

 if(isset($row['id'])) 
   {
         
$sql = "DELETE FROM `$name` WHERE id='".$row['id']."'";
echo $sql;
$query = mysql_query($sql) or die(mysql_error()); 
} 

[Benny-one]

wenn du kein echo erhälst, dann ist dein $row['id'] nicht gesetzt. Also stimmt was mit der SQL Abfrage ganz am Anfang nicht. Das mit dem $name ist vermutlich falsch bei deinem FROM.

[theaims]

Warum sollte $name falsch sein? Wird am anfang in der anfrage auch gesetzt. Und das sehe ich ja im Formular da stehen ja die ids.

Ich verstehe nicht ganz warum es nicht geht. ID muss doch gesetzt werden oder?

<form action="<?php echo $_SERVER['PHP_SELF'];?>" method="post"> 
<?php
$sql = "SELECT id FROM $name";
    $ergebnis = mysql_query($sql) or die(mysql_error());    
       if($ergebnis)
     {
       echo "<select name='id' size='1' onchange='document.forms.form1.an_id.value=this.form.id.options[this.form.id.selectedIndex].value'>";  
        
        echo "<option value='' selected></option>";
           while($row=mysql_fetch_array($ergebnis))
               {
               
               echo "<option value="."'".$row['id']."'".">".$row['id']."</option><br>";
                }
      echo "</select>";
      echo '<input type="submit" name="submit" value="Löschen">';
                           }
 if(isset($row['id'])) 
   {
         
$sql = "DELETE FROM `$name` WHERE id='".$row['id']."'";
echo $sql;
$query = mysql_query($sql) or die(mysql_error()); 
}

[Benny-one]

<?php
// Kommt mir komisch vor. Hat jeder $name eine eigene Tabelle?
$sql = "SELECT id FROM $name";
$ergebnis = mysql_query($sql) or die(mysql_error());    
if($ergebnis)
{
    // Wird das hier angezeigt?
    echo "<select name='id' size='1' onchange='document.forms.form1.an_id.value=this.form.id.options[this.form.id.selectedIndex].value'>";  
    
    echo "<option value='' selected></option>";
    while($row=mysql_fetch_array($ergebnis))
    {
        echo "<option value="."'".$row['id']."'".">".$row['id']."</option><br>";
    }
    echo "</select>";
    echo '<input type="submit" name="submit" value="Löschen">';
}

// Soll dieser Teil nicht vermutlich in die Schleife rein?
if(isset($row['id'])) 
{
    $sql = "DELETE FROM `$name` WHERE id='".$row['id']."'";
    echo $sql;
    $query = mysql_query($sql) or die(mysql_error()); 
}

?>

[theaims]

<?php
// Nein das mache ich nur für mich. Ist aber auch egal.

$sql = "SELECT id FROM $name";
$ergebnis = mysql_query($sql) or die(mysql_error());    
if($ergebnis)
{
    // Wird angezeigt
    echo "<select name='id' size='1' onchange='document.forms.form1.an_id.value=this.form.id.options[this.form.id.selectedIndex].value'>";  
    
    echo "<option value='' selected></option>";
    while($row=mysql_fetch_array($ergebnis))
    {
        echo "<option value="."'".$row['id']."'".">".$row['id']."</option><br>";
    }
    echo "</select>";
    echo '<input type="submit" name="submit" value="Löschen">';
}

// Warum denn? Er soll doch nur ausgeführt werden wenn man auf 
// Löschen klickt.
if(isset($row['id'])) 
{
    $sql = "DELETE FROM `$name` WHERE id='".$row['id']."'";
    echo $sql;
    $query = mysql_query($sql) or die(mysql_error()); 
}

?>

[Benny-one]

<?php
// Nein das mache ich nur für mich. Ist aber auch egal.

$sql = "SELECT id FROM $name";
$ergebnis = mysql_query($sql) or die(mysql_error());    
if($ergebnis)
{
    // Wird angezeigt
    echo "<select name='id' size='1' onchange='document.forms.form1.an_id.value=this.form.id.options[this.form.id.selectedIndex].value'>";  
    
    echo "<option value='' selected></option>";
    while($row=mysql_fetch_array($ergebnis))
    {
        echo "<option value="."'".$row['id']."'".">".$row['id']."</option><br>";
    }
    echo "</select>";
    echo '<input type="submit" name="submit" value="Löschen">';
}

// Dann bitte aber
if(isset($_POST['id'])) 
{
    $sql = "DELETE FROM `$name` WHERE id='".$_POST['id']."'";
    echo $sql;
    $query = mysql_query($sql) or die(mysql_error()); 
}

?>

[theaims]

Jetzt erhalte ich schon eine Ausgabe.

DELETE FROM `Test` WHERE id=''

Aber warum zum Teufel wird id nicht definiert?

[Benny-one]

füge das mal an Anfang deiner Datei

echo "<pre>";
print_r($_POST);
echo "</pre>"; 

und prüfe, ob ID vorhanden ist oder nicht.

[wahsaga]

Original geschrieben von theaims
Ich verstehe nicht ganz warum es nicht geht. ID muss doch gesetzt werden oder?

Denk mal nach, wann diese while-Schleife

while($row=mysql_fetch_array($ergebnis)) 

beendet wird, und was $row danach für einen Inhalt hat.

[Benny-one]

@wahsaga: das ist schon geklärt. Die ID solll von der Dropdownliste gesetzt werden und den jeweiligen Eintrag löschen.
Deswegen heißts auch mittlerweile $_POST['id'];