DELETE FROM werkt niet

Door Kristof s op 10-04-2007 23:00

1.392 views

heb lijstje gegenereerd met fototje om te deleten (dit werkt wel) buiten dat ie de titel herhaald maar dat komt omdat die binnen de while staat.

Maar ik wil nu als ik op dat fototje druk dat die de post met die id delete
hieronder code voor genereren van lijstje:

<?php
// aanmaken van pagina en tabel voor wijzigen van een bericht
if ($action == 'bewerk')
{

$sql= "SELECT NieuwsDatum,NieuwsOnderwerp,NieuwsText,NieuwsID,NieuwsGepostDoor FROM nieuws ORDER BY NieuwsID DESC ";
$query = mysql_query($sql) or die(mysql_error());
while($line = mysql_fetch_array($query,MYSQL_NUM))
{
$standaard = 'logo.gif';
$Nieuwsdatum = $line[0];
$NieuwsOnderwerp = $line[1];
$NieuwsText = $line[2];
$id = $line[3];
$nick = $line[4];
/* <tr bgcolor="#ECECEC" onmouseover="this.bgColor=\'#ff9900\'"
onmouseout="this.bgColor=\'#ECECEC\';this.style.cursor=\'pointer\'"
onclick="window.location=\'?page=nieuws&id='.$line[3].'&action=dobewerk\'"> */

echo'
<center>
<table cellpadding="0" cellspacing="0" width="500" border="0">
<tr>
<td width="10%" id="header">ID</td>
<td width="15%" id="header">Naam</td>
<td width="50%" align="center" id="header">Onderwerp</td>
<td width="100%" align="center" id="header">Datum</td>
<td width="10%" id="header"></td>
<td width="10%" id="header" ></td>
</tr>
<tr bgcolor="#2CC0E8" onmouseover="this.bgColor=\'#ff9900\'"
onmouseout="this.bgColor=\'#2CC0E8\';" >

<td id="kolrechts">'.$id.'</td>
<td align="left" id="kolrechts">'.$nick.'</td>
<td align="center" id="kolrechts">'.$NieuwsOnderwerp.'</td>
<td align="center" id="kolrechts">'.$Nieuwsdatum.'</td>
<td>
<a href="?page=nieuws&id='.$line[3].'&action=dobewerk">
<img src="afbeeldingen/knoppen/wijzig.gif" alt="wijzig" width="16" height="16" id="noborder">
<a/>
</td>
<td>
<a href="?page=nieuws&id='.$line[3].'&action=dodelete">
<img src="afbeeldingen/knoppen/delete.gif" alt="delete" width="16" height="16" id="noborder">
</a>
</td>
</tr>
</table>
</center>';
}

}

?>

en dit is query om te deleten, dit werkt NIET , iemand enig idee waarom ?

<?php

if($action == 'dodelete')
{
$sql2 = "DELETE FROM nieuws WHERE NieuwsID = '".$id."'";
$result = mysql_query($sql2) or die('Kan het nieuwsbericht niet verwijderen!');
echo 'Het nieuwsbericht werd succesvol verwijderd, u keert nu terug naar de site.
<meta http-equiv="refresh" content="1;URL=index.php" />';
}

?>

PHP Newbie

10-04-2007 23:09

Enkele dingen:

- Waar komt $id vandaan
- Het is een goede gewoonte om bij DELETE query's waarbij er maar 1 resultaat verwijderd moet worden, even LIMIT 1 erachter te zetten
- Als je even de mysql_error() echoot, dan weet je waarschijnlijk precies wat er verkeerd gaat

Kristof s

10-04-2007 23:13

de id komt uit de while lus
id = $line[3];

ik heb er hetvolgende van gemaakt
<?
$sql2 = "DELETE FROM nieuws WHERE NieuwsID = '".$id."' LIMIT 1";
$result = mysql_query($sql2) or die(mysql_error());
?>

maar ik krijg geen error, ik krijg mijn boodschapje dat hij zogezegd gedelete is, wat dus niet het geval blijkt te zijn.

Frank -

10-04-2007 23:18

mysql_affected_rows() gebruiken om te checken hoeveel er is verwijderd en de query echoen om te zien wat er richting database is gestuurd. Daar word je altijd een stuk wijzer van.

PHP Newbie

10-04-2007 23:21

Nog een tip, gebruik geen hoofdletters in je tabelnamen. Gewoon allemaal kleine letters houden, op die manier kun je ook nooit een fout maken met een hoofdletter die je ergens verkeerd hebt neergezet.

Voer dit eens uit:

<?php
$sql_select = mysql_query("SELECT id FROM nieuws WHERE NieuwsID = '".$id."' LIMIT 1");

if($sql_select)
{
echo 'SELECT Query met succes uitgevoerd, aantal rijen gevonden: '.mysql_num_rows($sql_select).'<br />';

$sql_delete = mysql_query("DELETE FROM nieuws WHERE NieuwsID = '".$id."' LIMIT 1");

if($sql_delete)
{
echo 'DELETE Query met succes uitgevoerd';
}
else
{
echo 'DELETE Query mislukt, error: '.mysql_error();
}
}
else
{
echo 'SELECT Query mislukt, error:'.mysql_error();
}
?>

Kristof s

10-04-2007 23:26

met volgende code :
<?
if($action == 'dodelete')
{
$sql2 = "DELETE FROM nieuws WHERE NieuwsID = '".$id."' LIMIT 1";
$result = mysql_query($sql2) or die (mysql_affected_rows()) ;
echo $result;
}

?>

dit is de url dat ik heb dus hij neemt de url en de action juist over
/localhost/site/nieuwsadmin.php?page=nieuws&id=105&action=dodelete

en hij retourneert mj van de echo
het getal 1

Kristof s

10-04-2007 23:32

heb net je script even toegepast (staat foutje in op lijn zes denk ik )
anyway, hij vind dus blijkbaar mijn lijn niet want ik krijg hetvolgende

SELECT Query met succes uitgevoerd, aantal rijen gevonden: 0
DELETE Query met succes uitgevoerd

best wel lullig :s

Frank -

10-04-2007 23:36

Superglobals...

Zoek daar maar eens op. Jouw script is in elk geval flink achterhaald en onveilig.

PHP Newbie

10-04-2007 23:50

Voilá, dan weet je waar het probleem zit. MySQL ziet de rij die jij wilt verwijderen niet eens, laat staan dat hij die rij kan verwijderen...

Kristof s

11-04-2007 00:25

nou momenteel delte is al wel, maar er is nog iets mis, hij delete enkel de meest recente. dus er is nog steeds iets mis met de id, alvast al erg bedankt voor jullie hulp

Kristof s

11-04-2007 01:05

het werkt, bedankt alletwee voor de uitleg
de superglobals hebben het gedaan :)
many thanks

Reageren

Inloggen om te reageren