Trying to delete row using pdo statement where row item matches php variable

Question

I am trying to receive text input and store it as a variable. Then I use PDO to execute a query that deletes a row from a MySQL database that contains text that matches the text stored in the variable. Obviously this is not working. My usual suspect is my query. I have also been toying with binding params but that didn't work either.

 <?php
     if (isset($_POST['gigDate'])){

            $gigDate  = $_POST['gigDate'];

               if (!empty($gigDate)){

                   require_once("dbconnect.php");
                   $query = $connect->prepare("DELETE * FROM 'gigs' WHERE 'date'='".$gigDate."'");
                   $query->execute(); 

                   }
               else 
                    {
                       echo "failure";
                      }
               require_once('gui.php');
                }
   ?>

This is my html form

   <h3>Delete a gig by date. (copy and paste gig date)</h3>

   <?php include ("gigDelete.php");?>

   <form action="gigDelete.php" method="post">
   <input type="text" name="gigDate"></input>
   <button type="submit" value="Delete" name="Delete">delete gig</button>
   </form>

I have had one error message that stated that my error reporting wasn't working. haha. So I removed it. I only just learned PDO last week and I am unsure if it is appropriate for this task.. How my inexperience shines!

This is the last part of my first ever php GUI. I can upload to the database just dandy and I am displaying the data in a html table which is also working. So I can assume I'm connecting to the db.

For one thing, you're using the wrong identifiers for both your table and column. Plus, get rid of the *. Read up on the function => dev.mysql.com/doc/refman/5.0/en/delete.html — Funk Forty Niner
– Funk Forty Niner, Commented Feb 4, 2015 at 15:06
What's the point of using prepared statements if you are hardcoding the values? — Nelson Galdeman Graziano
– Nelson Galdeman Graziano, Commented Feb 4, 2015 at 15:06
Add $connect->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); right after the connection is opened and you will see the errors. — Funk Forty Niner
– Funk Forty Niner, Commented Feb 4, 2015 at 15:10
The * is gone. I should point out that i am not on localhost. The site is live. I received a request to add this delete functionality. So I am uploading to test. Is this bad practice? — Forgive me I'm a drummer.
– Forgive me I'm a drummer., Commented Feb 4, 2015 at 15:20
Is it working from you now, as per the answers given? Most of which are incorrect. Bad practice? Your code is full of syntax errors that I've already pointed out. Have you looked into anything I've said above? — Funk Forty Niner
– Funk Forty Niner, Commented Feb 4, 2015 at 15:23

A l w a y s S u n n y · Accepted Answer · 2015-02-04 15:16:06Z

1

Try this way, why you are using extra single quotes for table and column names? The identifier quote character is the backtick (`)

DELETE FROM  `gigs` WHERE date='".$gigDate."'

edited Feb 4, 2015 at 15:16

answered Feb 4, 2015 at 15:09

A l w a y s S u n n y

38.8k9 gold badges69 silver badges120 bronze badges

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

2 Comments

Forgive me I'm a drummer. Over a year ago

I've didn't even know backticks existed until right now.

A l w a y s S u n n y Over a year ago

nevermind bro, it is enough to know that "Stackoverflow Exists..", ha ha best of luck

Nelson Galdeman Graziano · Accepted Answer · 2015-02-04 15:26:55Z

1

Try like this:

$query = $connect->prepare("DELETE FROM gigs WHERE date='".$gigDate."'");

Btw, if you are using PDO, there is no good reason to hardcode values like that.

edited Feb 4, 2015 at 15:26

answered Feb 4, 2015 at 15:07

Nelson Galdeman Graziano

5624 silver badges17 bronze badges

1 Comment

Funk Forty Niner Over a year ago

This will fail, being the * in DELETE * FROM

Joci93 · Accepted Answer · 2015-02-04 15:10:57Z

-1

I hope this will work for you:

$query = $connect->prepare("DELETE * FROM 'gigs' WHERE 'date'= :date");
$query->bindParam(':date', $gigDate);
$query->execute();

answered Feb 4, 2015 at 15:10

Joci93

8333 gold badges10 silver badges24 bronze badges

4 Comments

Forgive me I'm a drummer. Over a year ago

I'll give it a crack.

Forgive me I'm a drummer. Over a year ago

I tried it and no success. I can assume that the PDO is not the problem.

Funk Forty Niner Over a year ago

This will fail for two reasons.

Funk Forty Niner Over a year ago

Wrong identifiers dev.mysql.com/doc/refman/5.0/en/identifier-qualifiers.html and not the way to delete dev.mysql.com/doc/refman/5.0/en/delete.html - See the accepted answer.

Collectives™ on Stack Overflow

Trying to delete row using pdo statement where row item matches php variable

3 Answers 3

2 Comments

1 Comment

4 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

2 Comments

1 Comment

4 Comments

Your Answer

Sign up or log in

Post as a guest

Related