deleting multiple rows in a DB with checkboxs

[cferge]

Hey people how's it goin... I have a MAJOR PROBLEM... and would love for you experts to help me.. I know ur experts because I"ve been reading the posts...

Currently I'm trying to remove multiple rows of data with checkboxs and well... it's not working... 🙁 So I went out on the net and got some information on doing this... since I've got like more then a lil bit of PHP And SQL under my wing... butI still need help... here's the code Im using...

member.php:

<form name=sa_table_del action="list_delete.php" method=post>

echo "<td><input type=\"checkbox\" name=\"delete_list[]\" value=\"$listid\"></td></tr>";

echo "<tr><td colspan=2 align=center><input type=\"submit\" name=\"delete_list[]\" value=\"Delete\"></td></tr>";

delete_list.php:

$delete = "'" . implode( "', '", $delete_list );
$delete = substr( $delete, 0, ( strlen( $delete ) - 3 ) );
echo "$delete";

if(!($result = mysql_db_query($DB_CONNECT, "DELETE FROM SENT_ITEMS WHERE ID= '$delete'")))
{
DisplayErrMsg(sprintf("internal error %d:%s\n",
mysql_errno(), mysql_error()));
mysql_close($link);
exit();
}

Now here's the "CRAP" that I get back...

'', '', '', '', '', '', '', 'Del
Warning: MySQL Connection Failed: Access denied for user: 'ODBC@localhost' (Using password: NO) in d:\program files\apache group\apache\htdocs/list_delete.php on line 16

Warning: MySQL: A link to the server could not be established in d:\program files\apache group\apache\htdocs/list_delete.php on line 16

Fatal error: Call to undefined function: displayerrmsg() in d:\program files\apache group\apache\htdocs/list_delete.php on line 18

HELP ME PLEASE!!! I've read through the archives and I'm still stumped... Anymore information needed get ahold of me on AIM, I check it at 730pm EST... Thanks Alot! I hope someone can help me on this perdicament...

[notionlogic]

At first glance, this

Warning: MySQL Connection Failed: Access denied for user: 'ODBC@localhost' (Using password: NO)

would seem to be the problem

[cferge]

I know thats a problem, but whats the problem with the code? the authentication I can fix in a second... but when I do fix it I always get a parse error 🙁 just not fair... this is really starting to suck... 🙁

Thanks for the reply tho... atleast someone did... 🙂

[Chew]

I too am having the same dilemna...

Is there someone that can point me in the right direction on what I need to do to get this working?

Thanks,
Chew

[OctoberAsh]

Some things to check on, try:

1) Try echoing your $delete before you perform the implode, and see what data are being passed from your form initially.

2) What is it you want to do with the code below?

$delete = substr( $delete, 0, ( strlen( $delete ) - 3 ) ); 

What it's really doing is taking each element in the array and chopping off the last three characters. If all array elements are 3 chars or less, that would give you empty results as well.

3) Try commenting out your DisplayErrMsg() function, and see what the native error handling is returning (i.e. use mysql_err() and mysql_errno() only)

Good luck!

[ahundiak]

Assuming you solve your connection problem and assuming that you get $delete to be a comma delimited list of ids then

$delete = '10,20,30';
$sql = "DELETE FROM sent_items WHERE id IN ($delete)";
mysql_query($sql);

Using id = will not work with multiple id's.

[Chew]

Well, after some sleep, and food, I was able to get the code working this morning. I'll explain what they're doing below. I just want to give a big thanks to ahundiak, as his bit of code was one of the keys to getting it right.

Checkbox Display

<form name=sa_table_del action='test3.php' method='post'>
<input type='checkbox' name='delete_list[]' value='$row[postid]'>
<input type="submit" name="delete_list[]" value="Delete">

Delete Page

$delete = "" . implode( ", ", $delete_list );
$delete = substr( $delete, 0, ( strlen( $delete ) - 8 ) );

echo "$delete";

require "config.php";
$connection = @mysql_connect("$server", "$user", "$pass") or die("Could not Connect.");
$db = @mysql_select_db($dbname, $connection) or die("Could not select database.");
$sql = "DELETE FROM TABLENAME WHERE POSTID IN ($delete)";
$result = @($sql,$connection) or die("Could not execute query.");

As you can see, I totally omitted the DisplayErrMsg function, cause A) I didn't know what the hell it was doing, and 😎 Didn't see a need for it. Now the bit about the $delete = substr...

It seems that when you click the delete button (form submit), for some reason Delete is passed as a variable. So what the - 8 does is remove that from the list of variables. Also the formatting for php was all wrong, and after removing the misc 's he had placed everywhere, the script now works perfectly.

Thanks to everyone who solved a problem I've been plagued with for days, and to cferge for the original bit of code.

Questions? Leave them here, or email me. I'll respond to everyone who does.

Chew

[OctoberAsh]

Ok, it was really bugging me why you both were getting "Delete" passed into your queries, and I think I spotted the reason. You both use

<input type="submit" name="delete_list[]" value="Delete"> 

to create your "Submit" buttons. Try changing it to

<input type="submit" value="Submit">

and see if you still get anything tacked on to the end of your data. If it doesn't, you can eliminate the substr part of your code altogether.

Just a thought,