Deleting Multiple Rows using Checkboxes

[gismo]

Hi all,

I am having a lot of problems getting this working. The follwoing is the code that i am using:

printf("<tr><td><input type = checkbox name = \"delete[]\" value = \"<?=$myarray[$id]?>\"> </td></tr>, $myrow['name']);

<form method = "post" action="delete.php">
<input type="submit" name="delete" value = "delete">

Delete.php:

if(!$id]){
echo "<b>Please make your selection.</b>";
}//end if
else {
$sql = "DELETE FROM register WHERE id = '$myarray[$id]'";
$result = mysql_query($sql);
echo "<b>Information deleted.";
}//end else

Any help would greatly appreciated

Thank You
Gismo 🙂

[laserlight]

If the checkbox was checked, it's name/value would be passed.

So run a foreach loop, and delete all that is passed.

[mrhappiness]

or you have a look at mysql's IN operator

[gismo]

Hi

Thank you for the reply. I have tried an foreach loop as below but i am coming up with the following error

Warning: Invalid argument supplied for foreach() in c:\apache\htdocs\delete.php on line 13
0 rows deleted.

total.$deleted = 0;

foreach($_POST["delete"] as $id)
{
$sql = 'DELETE FROM register WHERE id='.$id;
if(($result = mysql_query($sql)) === FALSE)
{
exit('SQL Error:<br />'.$sql.'<br />'.mysql_error());
}

$deleted += mysql_affected_rows();
}

echo $deleted.' rows deleted.';

<input type = 'checkbox' name = 'delete[]' value = '$row[$id]'>

Any help is greatly appreciated
Thank You
Gismo

[laserlight]

if (isset($_POST['delete']) && is_array($_POST['delete']))
{
	$deleted = 0;
	foreach ($_POST['delete'] as $id)
	{
		if (mysql_query("DELETE FROM `register` WHERE `id`='$id'"))
		{
			$deleted += mysql_affected_rows();
		}
		else
		{
			die("SQL error:<br />" . mysql_error());
		}
	}
	echo $deleted . " rows deleted";
}

<input type="checkbox" name="delete[]" value="<?php echo $row[$id]; ?>">

[gismo]

Hi,

Tried that, but getting the following error:

Parse error: parse error in c:\apache\htdocs\results.php on line 46

for:

<input type="checkbox" name="delete[]" value="<?php echo $row[$id]; ?>">

i have tried changing it to single quotes but then i get an error on the other code.

Thanks
Gismo

[laserlight]

hmm... what is the code that you are using?

My example looks ok to me.

[gismo]

hi
Here is the code that im using-

$result = mysql_query("SELECT FirstName,LastName,address,telephone, DOB, email,username,password,confirmpassword FROM register",$db);

if ($myrow = mysql_fetch_array($result)) {

     echo "<table border=1 bordercolor= #66ccff>\n";



    echo "<tr><td>First Name</td><td>LastName</td><td>Address</td><td>Phone Number</td><td>Date of Birth</td><td>Email Address</td><td>User name</td><td>Password</td><td>Select</td></tr>\n";

do {

    printf("<tr><td>%s</td><td>%s</td><td>%s</td><td>%s</td><td>%s</td><td>%s</td><td>%s</td><td>%s</td><td><input type = 'checkbox' name = 'delete[]' value = '$row[$id]'></td></tr>\n", $myrow[0], $myrow[1], $myrow[2], $myrow[3], $myrow[4], $myrow[5], $myrow[6], $myrow[7]);

}while ($myrow = mysql_fetch_row($result));

} else {

     echo "Sorry, no records were found!";

}

echo "</table>\n";

?>

<center><form method = "post" action="delete.php">
<input type="submit" name="delete" value = "delete">

Delete.php:

if (isset($POST['delete']))
{
$deleted = 0;
foreach($POST['delete'] as $id)
{
if (mysql_query("DELETE FROM register WHERE id='$id'"))
{
$deleted += mysql_affected_rows();
}
else
{
die("SQL error:<br />" . mysql_error());
}
}
echo $deleted . " rows deleted";
}

?>

Thank You
Gismo

[laserlight]

Okay, you'll need to unlearn what you have learned :p

If you read this threadp, you would have noticed a problem with your style.

The next thing: place your code within [php] and [/php] bbcode formatting tags when posting.
By using separate bbcode blocks, you also make it clear which code belongs where.

[gismo]

Thanks but i am still a little confused.

Is it better not to use a while statement and $myrow = mysql_fetch_row($result)??

What would i use instead?

Thank You,
Gismo