jQuery Ajax delete button works but data not deleted in database

Question

The button delete is working fine it's just that it says that 'record deleted successfully' but it wasn't deleted at all at the table and database.

Here's my code for delete query

<?php require_once('process.php');
if(isset($_GET['id'])) {
$sql = "DELETE FROM tbl_oda_request WHERE id=".$_GET['id'];
$mysqli->query($sql);
echo 'Record deleted successfully.'; }?>

delete button

<td><button class='btn btn-danger delete'>Delete</button></td>

my script

<script type="text/javascript">
    $(".delete").click(function(){
        var id = $(this).parents("tr").attr("id");
        if(confirm('Are you sure to delete this record ?')) {
            $.ajax({
                url: 'delete_oda_req.php',
                type: 'GET',
                data: {id: id},
                error: function() {
                  alert('Something is wrong, couldnt delete record');
                },
                success: function(data) {
                    $("#" + id).remove();
                    alert("Record delete successfully.");  
                }
            });
        }
    });
</script>

I think you dont get the ID correctly in your php function, try to do echo $_GET['id'] and see if you get it — Burhan Kashour, Oct 26 '20 at 15:43
it's probably a problem with mysql connection. do you connect with $mysqli ? send your connection code with it's username and password removed. — Pooya Estakhri, Oct 26 '20 at 16:05
i tried to echo $_GET['id'] but the message still "Record delete successfully" but it wasn't deleted. — Nicole Rioveros, Oct 28 '20 at 12:28

score -1 · Answer 1 · answered Oct 26 '20 at 16:00

-1

Dump your $_GET variable. Has it an id?

Note: You should not add the id to the string. It cause security issues. see this post

answered Oct 26 '20 at 16:00

Dikkepanda

21
1
8

if id was empty then if would not run and echo wouldn't show up. – Pooya Estakhri Oct 26 '20 at 16:03

jQuery Ajax delete button works but data not deleted in database

1 Answers1