2

sou iniciante em php. alguém sabe por que não está funcionando? Ele faz o update corretamente de valores inteiros, mas se for uma string de letras não funciona

html Ele vai digitar aqui uma url pra fazer o update

<input type="text" id="url_video">

salva.php

$con = mysqli_connect("localhost","root","","database_name");

$resultado = mysqli_query($con,"Select * from video");

    echo "<table class='table'>".
        "<thead>".
             "<tr>".
                "<td>ID</td>".
                "<td>url_video</td>".
             "</tr>".
        "</thead><tbody>";
while($valor = mysqli_fetch_array($resultado)){
    echo "<tr>".
            "<td>".$valor['ID']."</td>".
            "<td>".$valor['url_video']."</td>".
            "<td><input type='button' value='Update' class='btn-update' data-id='".$valor['ID']."'>"."</td>".
         "</tr>";
}
echo "</tbody></table>";

ajax Eu mando a string digitada (url) e passo o id da coluna selecionada.

$(document).ready(function(){
    $('.btn-update').click(function(){
        var id = $(this).data('id');
        var string = $('#url_video').val();
        $.ajax({
            url: "tabelaUpdate.php",
            data: { 'idDeUpdate' : id,
                    'url_video' : string
                  },
            type: "POST",
            cache: false,
            success: function(response){
                $('#result').html(response); 
            }
        })
    });
});

php tabelaUpdate.php

$con = mysqli_connect("localhost","root", "", "database_name");
$id = $_POST['idDeUpdate'];
$string_url = $_POST['url_video'];


mysqli_query($con,"Update video SET url_video=$string_url WHERE ID=$id");
echo "ID -> ".$id." URL -> ".$string_url;
11
  • Só de curiosidade, por que você está usando supressão (@) no mysql_connect?
    – Largato
    26/03/2015 às 5:33
  • se eu não colocar, aparece essa mensagem-> Deprecated: mysql_connect(): The mysql extension is deprecated and will be removed in the future: use mysqli or PDO instead 26/03/2015 às 5:36
  • E não é melhor consertar do que esconder o alerta? Sugestão: aproveite que você está montando a aplicação e já troque isso por mysqli, em vez de ter que refazer tudo quando removerem essa função do PHP. (inclusive por que hoje você está correndo um risco de injection nesse AJAX, pela maneira que faz o update).
    – Largato
    26/03/2015 às 5:37
  • Fiz algumas alterações no código usando mysqli. 26/03/2015 às 5:50
  • É bom que vc tenha posto o salva.php também. Pode ajudar alguém a achar o problema. Isso aqui me parece problemático: var string = $('#url_video').val();, vai sempre pegar o 1o URL, não?
    – Largato
    26/03/2015 às 5:53

1 Resposta 1

3

Eu sugeriria trocar esta linha:

"<td><input type='button' value='Update' class='btn-update' data-id='".$valor['ID']."'>"."</td>"

por esta:

"<td><input type='button' value='Update' class='btn-update' data-id='".$valor['ID']."' data-urlvideo='".$valor['url_video']."'>"."</td>"

E no JS pegar os dois parâmetros usando .data:

  var id = $(this).data('id');
  var string = $(this).data('urlvideo');

Cabe também, como sugerido pelo @Fernando, checar se a tabela do DB está com o tipo correto no campo url_video, que deve ser textual, como char ou varchar.

Para completar, eu trocaria esta linha por "bind": em vez de usar concatenação de strings (assim você resolve o problema das aspas faltando, e se protege de injection):

Substituir:

mysqli_query($con,"Update video SET url_video=$string_url WHERE ID=$id");

Por:

$stmt = mysqli_prepare($con,"Update video SET url_video=? WHERE ID=?");
mysqli_stmt_bind_param($stmt, 'si', $string_url, $id);
$stmt->execute();

Assim estará minimizando a chance de SQL Injection.

Você deve fazer log-in para responder a esta pergunta.

Esta não é a resposta que você está procurando? Pesquise outras perguntas com a tag .