Delete PHP (mysqli) row - with HTML/Javascript button

Question

Using google maps, I have events saving to a database using mysqli. These events are then displayed as markers on the map and when clicked the relevant data is displayed in an info box (Name, date, etc). I want the option to delete an event event by deleting a row from the DB when the Remove (remove-event) button is clicked. The button is contained in the data displayed with the javascript:

var eventContent = $('<div class="event-info">' + '<h4 class="event-name">' +     point.name + '</h4><hr>' +
        '<span><h5>Date: </h5>' +
        '<p class="event-date">' + point.edate + '</p></span>' +
        '<p class="event-description">'+point.description+'</p>' +
        '</span><button id="remove-event" name="remove-event" class="remove-event btn btn-danger btn-sm" onclick="tidy_maps.delete()" title="Remove Event">Remove Event</button>'+
        '</div>');

    // Display Event details on marker click
    google.maps.event.addListener(event_markers[i], "click", function () {
        infowindow.setContent(eventContent[0]);
        infowindow.open(map, event_markers[i]);

The script that sends it to the php (removedata.php):

tidy_maps.delete = function() {
    $.ajax({
        type:'POST',
        url:'removedata.php',
        success:function(data) {
            if(data) {
                alert("Are you sure?");
            }
            else {
                alert("ERROR!!!!");
            }
        }
    });
}

The removedata.php is:

$con = mysqli_connect("localhost", "root", "password", "gmaps1");
if (!$con) {
die("Can not connect: " .mysql_error());
}

$sql = "DELETE FROM events WHERE id = 'id' ";

$query = mysqli_query($con, $sql);

if(mysqli_affected_rows($con)) {
    echo "Record deleted successfully";
}

mysqli_close($con);

As it is, it does not delete the row in the DB, but when i change the line:

$sql = "DELETE FROM events WHERE id = 'id' ";

to a specific ID No. Example:

$sql = "DELETE FROM events WHERE id = '5' ";

And i run the removedata.php in the browser, it deletes the row with ID=5 from the DB. There seems to be no errors when the console when clicking the remove button so it must be sending to PHP script ok.

I would like when the Remove button is clicked that it asks are you sure and then it deletes that specific Row form the DB.

You forgot the $ sign in the id. "DELETE FROM events WHERE id = $id "; — kidA
– kidA, Commented Mar 24, 2015 at 22:41
Assign the row id to a variable ... $id = 5; $sql = "DELETE FROM events WHERE id = '$id' "; — Brian
– Brian, Commented Mar 24, 2015 at 22:41
Where do you pass the id? success:function() is called if the request was successful - you need to show a confirm-box before you make the request. — msfoster
– msfoster, Commented Mar 24, 2015 at 22:51
I don't see any ID or Data passing from client to server? So how can it be delete? — Blu
– Blu, Commented Mar 25, 2015 at 14:03

msfoster · Accepted Answer · 2015-03-25 13:47:02Z

1

As far as I can tell you don't pass the ID of the row to be deleted. You can send data two ways, either as a url parameter, or post it using the data tag:

$.ajax({
    type:'POST',
    url:'removedata.php',
    data: {id : 5}
});

Access the ID in removedata.php:

$id = intval($_POST["id"]);
$sql = "DELETE FROM events WHERE id = " . $id;

answered Mar 25, 2015 at 13:47

msfoster

2,5721 gold badge19 silver badges19 bronze badges

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

2 Comments

Stephen Over a year ago

I have tried accessing ID in removedata.php but it just gives an error of: "Notice: Undefined index: id in/......./ on line 12"

msfoster Over a year ago

Please show the code that you have tried. You have to make the request, if you go to removedata.php manually the ID will not be sent.

Alex R · Accepted Answer · 2015-03-24 22:44:02Z

0

WHERE id = 'id' you need to remove the '' and add the $ symbol if you want id to be a variable.

answered Mar 24, 2015 at 22:44

Alex R

451 bronze badge

2 Comments

Stephen Over a year ago

This doesnt quiet solve it. It still remains in the DB. Should I be passing "id" in the "tidy_maps.delete = function()"??

Alex R Over a year ago

You could pass it using your ajax request with the help of the data tag. api.jquery.com/jquery.ajax

Stephen · Accepted Answer · 2015-03-24 23:33:04Z

0

Ok I've played around a little and amended the JS slightly:

tidy_maps.delete = function() {
var confirm_remove = confirm("Do You Want to Remove This Event?")
if(confirm_remove) {
    $.ajax({
        type:'POST',
        url:'removedata.php',
    });
    window.location = "http://www.google.com/";
}
else {
    alert("ERROR!!!!");
}
}

So when Confirm is YES, i threw in a redirect to Google just to see what happens. When YES is clicked in the confirm box, it redirects the page to Google but does not delete the row from the DB

answered Mar 24, 2015 at 23:33

Stephen

5372 gold badges7 silver badges20 bronze badges

4 Comments

Stephen Over a year ago

No, just looking at it now again. It seems to be going to the removedata.php script but that itself does not delete the intended row. I'm just looking at another way, by sending the lat and lon of the map marker and using them instead of id

msfoster Over a year ago

Well, do you pass the id to removedata.php?

Alejandro Iván Over a year ago

type: 'POST', url: 'removedata.php', data: { id: <the_id_you_want_to_remove> }, success: function() { ... }

Stephen Over a year ago

the thing is, the ID being sent will be that of whatever marker is being accessed. So instead of using a numerical value like'5', i need it to be a variable, so should the ID be sent like: "data: {id: $id}" ?

Vijay · Accepted Answer · 2015-03-26 05:25:03Z

0

Try this

var id = 5;
var request = $.ajax({
  url:'removedata.php',
  type: "POST",
  data: "id="+id,
  success: function(data){
     console.log(data);
  }
});

get post value in removedata.php

 //get post value
 $id = intval($_POST["id"]);
 $sql = "DELETE FROM events WHERE id = " . $id;

answered Mar 26, 2015 at 5:25

Vijay

4313 silver badges12 bronze badges

1 Comment

Stephen Over a year ago

No because that will be sending the ID as 5 and deleting only that markers row. I want ID to be a variable so it deletes what ever event marker that is selected.

Collectives™ on Stack Overflow

Delete PHP (mysqli) row - with HTML/Javascript button

4 Answers 4

2 Comments

2 Comments

4 Comments

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

2 Comments

2 Comments

4 Comments

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related