Problem with post/get method to update table

Question

I’m trying to update my table using javascript so I don’t have to leave the screen every time I turn off a field, my table:

The button I used in PHP but went to another page:

<td class="center">

    <?php echo"<a class='btn btn-danger' href='update_produto.php?ID=$id'>Inativar</a>";?>

</td>

My button:

<button id='botao-olho' onclick='myFunction()'>Inativar</button>"

Here is the script for the update

<script type="text/javascript">

var btn_olho = $("#botao-olho");

btn_olho.on("click", function(){
    $.ajax({
        url: "update_produto.php",
        type:"html",
        dataType: "GET",
        data: {id: <?php echo $id ?>}
    }).done(function(retorno){

    });

});

here the update:

$id = $_GET['ID'];
$estado = 'Inativo';
$sql = "UPDATE MASTER_COLETORES SET ESTADO = '$estado' WHERE ID = $id";
$stmt = oci_parse($conexao, $sql);
$result = oci_execute($stmt);

I click the button and nothing happens :T

Answer 1

You can’t use the same id on all buttons. A id should be unique on the page. So change the id for class and remove the onclick which has no function:

<button class='botao-olho'>Inativar</button>

And add a data-id to get the id you want to send by clicking button, concatenating the variable $id of PHP:

<?php echo "<button data-id='".$id."' class='botao-olho'> Inativar </button>"; ?>

Now Ajax you will use only:

$(".botao-olho").on("click", function(){
   var $this = $(this); // salva o elemento numa variável
    $.ajax({
        url: "update_produto.php",
        data: {id: $this.data("id")}, // envia o id
        success: function(){
           $this.closest("tr").remove(); // remove a linha se ocorreu tudo ok
        }
    });
});

In PHP requested by Ajax you put the ID in lower case in GET:

$id = $_GET['id'];

In the Ajax options you have reversed the values of type and dataType, but you don’t even need these options since by default Ajax sends via GET and you don’t expect any feedback, just send the id. You don’t need the .done, just one success to remove the line where the button is.