DELETE Schleife mit $_REQUEST und Checkboxen

[surf_tiki]

Ich hoffe Ihr könnt mir helfen:

Ich habe eine Liste von Headlines aus der Datenbank generiert und dazugehörige Checkboxen. Wenn man diese Checkboxen anwählt und das Formular abschickt, sollen die markierten aus der Datenbank gelöscht werden. Jede Checkbox ist mit der dazugehörigen Headline Id gekennzeichnet. Funktionierte ganz toll, bis der WebAuftritt auf den neuen Server geschaufelt wurde. Klare Sache: register_globals=off... Doch wie krieg ich mein Array wieder hin:


Checkbox:

if ($_REQUEST["action"]=='delete') { 

mein Delete Script:

if ($_REQUEST["action"]=='delete') {
    $countDelete=count($_REQUEST["deleteBox"]);
    $whereClause="WHERE news_id='".$_REQUEST["deleteBox"][0]."'";
    
    for($i=1;$i<$countDelete;$i++){
        $whereClause.=" OR news_id=".$_REQUEST["deleteBox"][$i].";";
    }

    $query="DELETE FROM News ".$whereClause;
    
    $query=mysql_query($query)
    or die (mysql_error());
} 

Wie krieg ich das zum Laufen?

HILFE!!!

[Abraxax]

so sollte es optimal laufen.
bin auch von headline_id als id-feld ausgegangen.
( müsstest du evtl. anpassen. )


Checkbox im html in einer ausgabeschleife. ist ja klar...:

... name="deleteBox[]" value="<? echo $news_id?>" ....

das formular wird via post übergeben.

$_REQUEST nehmen wir mal nicht...

Dein Delete Script:

<?php

    if (isset($_POST['action']) && $_POST['action']=='delete' && isset($_POST['deleteBox']))
    {
        if (count($_POST['deleteBox']) > 0)
        {
            $sql = 'DELETE FROM News WHERE headline_id IN ('.implode(',', $_POST['deleteBox']).')';
            $res = mysql_query($sql) or die (mysql_error());
        }
    }

?>

[wahsaga]

Re: DELETE Schleife mit $_REQUEST und Checkboxen

$_REQUEST[] ist veraltet, bei einer neueren php-version benutzt du stattdessen $_GET[] oder $_POST[] (je nachdem, welche übertragungsmethode dein formular nutzt).

[surf_tiki]

SQL ERROR

Ihr seid ja wahnsinnig schnell! Danke erstmal!

Das Script scheint mir einwandfrei zu sein, doch bekomme ich einen

You have an error in your SQL syntax near ')' at line 1

???

[Coragon Rivito]

$_REQUEST ist NICHT veraltet, im gegenteil, gehört zu den neueren superglobals

es vereint einfach get, post, session, cookie (noch was ? *g*)

ich zb verwende $_REQUEST dann, wenn ich getrost davon ausgehen kann dass nix passiert ..

domain.tld/?logout=1

<?php
if (isset($_REQUEST['logout']) { /* etc */ }
?>

[Abraxax]

Re: SQL ERROR

Original geschrieben von surf_tiki
You have an error in your SQL syntax near ')' at line 1

ich habs nicht getestet....

// gib mal nach
$sql = "DELETE FROM..... ";

// das hier aus
echo '<pre>'.$sql.'</pre>'; 

ausgabe?

[surf_tiki]

DELETE FROM News WHERE news_id IN () 

[Abraxax]

hast du mein script 1:1 übernommen , oder änderungen gemacht?

eigentlich prüft das script alles ab, um diesen fehler nicht zu produzieren.

[surf_tiki]

ich hab das script 1:1 übernommen.
Hab nur die headline_id in news_id umbenannt. ...

[Abraxax]

wie sieht das html-formular aus?

[surf_tiki]

danke dass du dir die zeit nimmst

<form action="left.php" method="post" name="newsform"  enctype="multipart/form-data">
<input type="hidden" name="action" value="delete">

<table cellpadding="0" cellspacing="0" border="0">
    <tr>
        <td valign="top">
            <table width="330" cellpadding="0" cellspacing="0" border="0">
<?
/*/////////////////////////////////
// News Liste                    //
*//////////////////////////////////

$query_news=mysql_query("SELECT * FROM News ORDER BY erstell_datum DESC")
or die (mysql_error());
$num_rows=mysql_num_rows($query_news);
if ($num_rows!=0) {
    while ($news=mysql_fetch_array($query_news)) {
        $erstell_datum=ts2datum($news["erstell_datum"]);
        $news_id_liste=$news["news_id"];
        $headline_liste=$news["headline"];
?>
                <tr>
                    <td><input type="checkbox" class="check" value="<? echo $news_id?>" name="deleteBox[]"></td>
                    <td><img src="../../grafik/pixel.gif" width="5" height="1" border="0"></td>
                    <td><strong><? echo $headline_liste?></strong> / <? echo $erstell_datum ?> / <a href="middle.php?action=edit&news_id=<?echo $news_id_liste?>" target="middle">editieren</a></td>
                </tr>
<?
    }
}
?>
<tr>
                    <td>&nbsp;</td>
                    <td><img src="../../grafik/quader2.gif" border="0"></td>
                    <td><a href="javascript:deleteNews()">Auswahl l&ouml;schen</a></td>
                </tr>
</table>
</form>

[Abraxax]

mhm.... mache mal das am anfang der datei left.php rein.

echo '<pre>';
print_r($_POST['deleteBox']);
echo '</pre>'; 

ausgabe?

[surf_tiki]

Array
(
    [0] => 
    [1] => 
) 

[Abraxax]

aaarg.

ich habe noch einen fehler in deinem form-generier-script bei der checkbox gesehen.

dort gibt du ja kein value aus....


<? echo $news_id?>
$news_id gibt es nicht.

entweder $news_id_liste oder lieber direkt $news["news_id"] verwenden.

bitte tauschen und neue infos posten...

[surf_tiki]

Ich bin ein idiot

Danke dir..jetzt geht´s. Hatte gestern beim rumprobieren schonmal die id geändert aber dann wohl zu oft <strg>+z und nicht mehr dran gedacht---ich danke dir.