Deleting MySQL rows using checkboxes

calvin_leonhart

I have this code and it works.

        function del_admins($id, $del) {

            include '../lib/config.php';

            connect();

	$result = mysql_query("SELECT * FROM admin");
	$total_admins = mysql_num_rows($result);

            if ($total_admins == "1") {
            	CPMessage("Error Deleting Administrator", "Currently there is only one administrator, so the account cannot be removed.");
            }

            if ($del == "") {
            	main();
            }

            foreach($del as $id){
		$result = mysql_query ("DELETE FROM `admin` WHERE `id` = '".$id."'");
            }

            CPMessage("Administrator(s) Deleted", "The selected administrator(s) has been deleted.");

    }

Okay so my problem?

Well everything works fine. It doesn't let me delete rows if there is one left, which is the way I coded it. The problem-- if I try to delete all of them at the same time, it deletes them all. How do I prevent that from happening?

Rodney_H_

maybe use a call to the return() function after this line of code:

if ($total_admins == "1") 
{
   CPMessage("Error Deleting Administrator", "There is only one admin...");
   return;
}

... else the other parts of your user-defined function may still be executing...

HTH

TheDefender

calvin_leonhart wrote:
Okay so my problem?

Well everything works fine. It doesn't let me delete rows if there is one left, which is the way I coded it. The problem-- if I try to delete all of them at the same time, it deletes them all. How do I prevent that from happening?

You say the problem is if you try to delete all at the same time, it deletes them all. This would lead me to believe it's doing exactly what you want it to do...

Now, I'm guessing you may mean if you try to delete one, then all of them delete; or if you try to delete all, then only one deletes...

Either way, post your Form code. That will help a lot.

calvin_leonhart

I just want to prevent the code from deleting all rows at the same time if all checkboxes are checked. Other than that, it works the way I want it to.

Rodney_H_

Did you try adding the return function like I suggested, Calvin? Did that help??

calvin_leonhart

No.

Like I said, I don't want the script to delete all admins if all checkboxes are checked. And I have no idea how to achive that.

Rodney_H_

Well try what I suggested. I understood WHAT you wanted to achieve, and that is why I recommended it, Calvin.... try it, please.

calvin_leonhart

I did, but it didn't do anything.

Rodney_H_

Well, here is an idea:

Why not create a new field in your admin db table called: "default" where the field type was tinyint and the default value was "0".

That way, you would appoint one administrator in the system as the "default" and would be assigned a value of "1".

When you run your delete routine and query, you could simply:

// just concept, not actual code...
$query = "DELETE FROM admin_table WHERE default != 1 ";