deleting multiple entry in database using checkbox php pdo

Question

I am showing a list in my project the data is from a database i use php pdo to get the data now i want to have a function to delete multiple entry using checkbox i am doing this tutorial i am doing the answer of john, but im getting an error saying

Warning: PDOStatement::execute() expects parameter 1 to be array, string given in line $stmt->execute($id);

this is the entire code

function ImageGalleryDelete(){
    global $dbh;
    if(!empty($_POST['checkbox'])){
        $bigimage = $_POST['checkbox'];
        $stmt = $dbh->prepare("DELETE FROM imagegallery WHERE id = ?");
        foreach ($bigimage as $id)
            $stmt->execute($id);

    }else{
        echo "<script type='text/javascript'>alert('big image is empty');
                window.location='dashboard.php';
                </script>";
                exit;
    }       
}

why am i getting that error? Any Help will be appreciated..

why am i getting that error - because $id is not an array. Capt. Obvious. — u_mulder
– u_mulder, Commented Sep 24, 2014 at 8:51
so you mean to say i should put it like this $stmt->execute($bigimage ); bigimage here is array right? — Giant
– Giant, Commented Sep 24, 2014 at 8:52
....... Please read the manual entry for PDOStatement::execute() — Madara's Ghost
– Madara's Ghost, Commented Sep 24, 2014 at 8:53
$bigimage is array, but it doesn't suit. Read php manual, it's described there - php.net/manual/en/pdostatement.execute.php, example 3 explains everything — u_mulder
– u_mulder, Commented Sep 24, 2014 at 8:54

Madara's Ghost · Accepted Answer · 2014-09-24 09:01:03Z

2

The hint is already in the error message shown, feed an array with the correct format:

$bigimage = $_POST['checkbox'];
$stmt = $dbh->prepare("DELETE FROM imagegallery WHERE id = :id");
                                                       //   ^ named placeholder
foreach ($bigimage as $id) {
    $stmt->execute(array(':id' => $id));
                // ^ put an key value pair array inside with the designated named placeholder
                // along with the value
}

edited Sep 24, 2014 at 9:01

Madara's Ghost

176k52 gold badges274 silver badges314 bronze badges

answered Sep 24, 2014 at 8:59

Kevin

41.9k12 gold badges57 silver badges72 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

andyroo · Accepted Answer · 2014-09-24 09:03:07Z

0

The way I'd typically write this:

$stmt = $dbh->prepare("DELETE FROM imagegallery WHERE id = :id");
foreach ($bigimage as $id) {
    $stmt->execute(array(":id" => $id));
}

edited Sep 24, 2014 at 9:03

answered Sep 24, 2014 at 8:56

andyroo

3942 silver badges13 bronze badges

3 Comments

Madara's Ghost Over a year ago

I'm tempted to downvote you just for the omitted {}.

Madara's Ghost Over a year ago

And the other answer fixed it. You wouldn't leave an SQL injection vulnerability in code, would you? Why leave other glaring bad practices?

andyroo Over a year ago

I also wouldn't assume $_POST['checkbox'] is sanitized, just a little out of context for the question

Collectives™ on Stack Overflow

deleting multiple entry in database using checkbox php pdo

2 Answers 2

Comments

3 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

3 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related