[smithygotlost]

Heya guys

Ok this is probally simple but ive been awake for nearly 48 hours and cant think about it and google isnt helping much either, Im gonna be posting a list of id's to a script for now called example.php what i want to do is get everyone of these id, and update a database field with it, however the ammount submitted will never be the same. could be between 1 - 100 id's.

Its to mass delete the message's in a message center from check boxes.

so heres what i got

on the message page

PHP Code:

<input name="list" id="4775" value="4775" type="checkbox"> 

when submitted it runs this

PHP Code:

if($_POST["submit"] == "delete"){

    echo" submitted";

$message_delete = mysql_query("select * from `messages` where `toid`='$players_id' and `deleted`='0'")or die(mysql_error());

    while($row=mysql_fetch_array($message_delete)){

        $messageid = $row["messageid"];

        $toname = $row["toname"];

        $deletedmessageid = safe($_POST["list"]);

        echo" $deletedmessageid";

            mysql_query("update `messages` set `deleted`='1' where `messageid`='$deletemessageid'")or die(mysql_error());

            $sucess = "Message id# $deletemessageid Deleted";

    }

} 

Now its echo'in the same Message id and i cant work out why, The values in the check boxes on the previous page are all right but this script seems to just not wanna work.

Any ideas ?

Cheers
Mike

[Nico]

HTML Code:

 <input name="list[]" id="4775" value="4775" type="checkbox">  

(Note the square brackets)

PHP Code:

/* .... */



$deletedmessageid = array_map('intval', (array) $_POST['list']);

$deletedmessageid = implode(', ', $deletedmessageid);



if ($deletedmessageid)

{

    mysql_query("update `messages` set `deleted`='1' where `messageid` IN({$deletemessageid})")

        or die(mysql_error());



    $sucess = "Message id# $deletemessageid Deleted";

} 

[smithygotlost]

Thanks for that nico, But i get a syntax error :S weird

this is before

PHP Code:

echo"<input name=\"list[]\" id=\"$message_id\" value=\"$message_id\" type=\"checkbox\">"; 

then the bit you gave nico

PHP Code:

if($_POST["submit"] == "delete"){
$deletedmessageid = array_map('intval', (array) $_POST['list']);
$deletedmessageid = implode(', ', $deletedmessageid);

if ($deletedmessageid)
{
    mysql_query("update `messages` set `deleted`='1' where `messageid` IN({$deletemessageid}) and `toid`='$player_id'")
        or die(mysql_error());

    $sucess = "Message id# $deletemessageid Deleted";
}  
} 

and i get this error

PHP Code:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ')' at line 1 

its really weird, have i missed something :S

Cheers
Mike

[Nico]

What output does this give you:

PHP Code:

if($_POST["submit"] == "delete")

{

    $deletedmessageid = array_map('intval', (array) $_POST['list']);

    $deletedmessageid = implode(', ', $deletedmessageid);

    

    echo print_r($_POST['list'], true), "<br />\n";

    echo $deletedmessageid;

} 

...?

[smithygotlost]

PHP Code:

Array ( [0] => 12114 [1] => 12084 [2] => 12069 [3] => 11979 [4] => 11976 [5] => 11974 )
12114, 12084, 12069, 11979, 11976, 11974 

[smithygotlost]

Sorry forgot to add your help is appriciated nico

[smithygotlost]

Any more help nico ?

[End User]

In addition to all the other comments, I would add that technically an "id" can't begin with a number, so something like 'id="4775"' actually isn't legal. Most browsers will (might) handle it, but it's not valid HTML.

ID naming rules:

* Must begin with a letter A-Z or a-z
* Can be followed by: letters (A-Za-z), digits (0-9), hyphens ("-"), underscores ("_"), colons (":"), and periods (".")
* Values are case-sensitive

[smithygotlost]

Heya guys found the problem

PHP Code:

/* .... */

$deletedmessageid = array_map('intval', (array) $_POST['list']);
$deletedmessageid = implode(', ', $deletedmessageid);

if ($deletedmessageid)
{
    mysql_query("update `messages` set `deleted`='1' where `messageid` IN({$deletemessageid})")
        or die(mysql_error());

    $sucess = "Message id# $deletemessageid Deleted";
} 

In the sql statement it needed to say $deleted rather than $delete

Thanks Both Of You Much Appriciated