[SOLUCIONADO] Botón para eliminar el registro actual de la tabla

[rodrypaladin]

Buenas compis, no se si por el título me habréis entendido bien, he intentado expresar lo que quiero hacer, pero tal vez no lo haya conseguido.

con el siguiente código muestro los registros de una tabla.

Código PHP:

<?php
            //conexion se llama arriba con include a conexion.php
            
             //consulta
             $query ="SELECT * FROM PANEL ORDER BY id DESC" or die("Error en la consulta. " . mysqli_error($link));
             //ejecuta el query
             $resultado = mysqli_query($link, $query);
 
             while($row = mysqli_fetch_array($resultado)) {
             echo "<div class='box-grey'>";
             echo "<autor>Enviado por <font color='blue'><b>" . $row["usuario"] . "</b></font> el " . $row["fecha"] . "</autor>";
             echo "<br/>" . $row["texto"];
             echo "</div><br/>";
             ?>
 

hasta aquí bien, ahora lo que intento hacer es que justo igual que se muestra cada registro de forma correcta, también muestre en cada uno de ellos un botón con el que pueda borrar ese registro, es decir, mostraría un registro y un botón con el que podría eliminarlo.

Supongo que añadiendo un formulario con un botón tipo submit podría valer ( si no supongo mal ) pero no se como leches hacer para que elimine ese registro en cuestión.

Lo que yo he intentado es lo siguiente:

Código PHP:

<?php
            //conexion se llama arriba con include a conexion.php
            
             //consulta
             $query ="SELECT * FROM PANEL ORDER BY id DESC" or die("Error en la consulta. " . mysqli_error($link));
             //ejecuta el query
             $resultado = mysqli_query($link, $query);
 
             while($row = mysqli_fetch_array($resultado)) {
             echo "<div class='box-grey'>";
             echo "<autor>Enviado por <font color='blue'><b>" . $row["usuario"] . "</b></font> el " . $row["fecha"] . "</autor>";
             echo "<br/>" . $row["texto"];
             echo "<br/>";
             ?>
             <form method="POST" action="panel.php">
             <input type="submit" id="enviar2" name="enviar2" value="Borrar"/>
             <?php
             if(isset($_POST["enviar2"])) {
                $eliminar = "DELETE FROM PANEL WHERE id='" . $row["id"] . "'";
                $resultado_eliminar = mysqli_query($link, $eliminar);
             }
            echo "</div>";
           }
    ?>
 

Pero no funciona, le doy al botón que supuestamente borra pero no pasa nada. Aclaro que la tabla si está en mayúsculas ( PANEL ) a ver si podéis echarme un cable. Mil GRACIAS de antemano

[pateketrueke]

Debes separar toda la lógica primero, no es correcto realizar acciones dentro de un bucle.

Además lo correcto es usar un sólo formulario, no uno por cada fila, donde por la fila en cuestión debes usar un botón con un name adecuado:

Código PHP:

<?php
// acciones, previas...
 
if (!empty($_POST['borrar'])) {
  var_dump($_POST['borrar']);
}
 
// ...
 
?><form>
<?php while ... ?>
<button name="borrar[<?php echo $row->id; ?>]">borrar</button>
<?php ... ?>
</form>
 

La idea, es que name[N] sea tipo array y su clave represente el registro a borrar.

[rodrypaladin]

@pateketrueke me parece que crees que se más php del que realemnte se, jajaja.

Lo que tengo hasta ahora es lo siguiente:

Código PHP:

<?php
            //conexion se llama arriba con include a conexion.php
            
             //consulta
             $query ="SELECT * FROM PANEL ORDER BY id DESC" or die("Error en la consulta. " . mysqli_error($link));
             //ejecuta el query
             $resultado = mysqli_query($link, $query);
             ?>
             <form method="POST" action="panel.php">
            <?php
             while($row = mysqli_fetch_array($resultado)) {
             echo "<div class='box-grey'>";
             echo "<autor>Enviado por <font color='blue'><b>" . $row["usuario"] . "</b></font> el " . $row["fecha"] . "</autor>";
             echo "<br/>" . $row["texto"];
             echo "<br/>";
             ?>
             <input type="submit" id="enviar2" name="enviar2" value="Borrar"/>
             <?php
             if(isset($_POST["enviar2"])) {
                $eliminar = "DELETE FROM PANEL WHERE (id='" . $row['id'] . "')";
                $resultado_eliminar = mysqli_query($link, $eliminar);
                header('Location: panel.php');
             }
            echo "</div>";
           }
    ?>
    </form>
 

Aparece corectamente un botón en cada registro de la bd, pero cuando le doy a uno de ellos, tendría que borrar únicamente ese registro, pero los borra todos.

[chivacker]

Estoy de acuerdo con pateketrueke en que según vayas aprendiendo debes poco a poco separar la lógica de la visualización.

Dicho esto creo que la solución es simple. Yo lo haría al menos en 2 páginas distintas. En la primera dentro del while al final de mostrar los datos añade esto:

Código:

<form action="borrar_registro.php" method="post">
<input name="borrar" type="hidden" value="<?= row['id'] ?>" />
<input type="submit" value="Borrar Registro" />
</form>

Y en la página de borrar_registro.php esto:

Código:

<?php


if(!isset($_POST['borrar'])){
$id = $_POST['borrar'];
$eliminar = "DELETE FROM PANEL WHERE id='$id'";
$resultado_eliminar = mysqli_query($link, $eliminar);
}

Creo que se ha entendido.

Mucha suerte, saludos

[rodrypaladin]

Pues me da que no lo he entendido, lo estoy realizando en una sola página porque es como lo necesito, es un mini panel de administrador en el que manejo todo desde una página. Lo estoy intentando realizar en una página pero no lo consigo. Siento mi ignorancia

Por si no me explico bien, esto es lo que intento: imagen Al darle a borrar en cualquiera de los registros me borra todos!!

[pateketrueke]

Iniciado por rodrypaladin

Al darle a borrar en cualquiera de los registros me borra todos!!

Pues mientras tengas dicha instrucción dentro del while() seguirá pasando lo mismo.

Y vamos, sí es posible usar un único archivo, pero deberías separar las operaciones de la visualización.

[rodrypaladin]

Ya he sacado la instrucción del while() pero ahora en vez de borrar todos no elimina ninguno XD

Código PHP:

<?php
             $query ="SELECT * FROM PANEL ORDER BY id DESC" or die("Error en la consulta. " . mysqli_error($link));
             $resultado = mysqli_query($link, $query);
             ?>
             <form method="POST" action="panel.php">
            <?php
             while($row = mysqli_fetch_array($resultado)) {
             echo "<div class='box-grey'>";
             echo "<autor>Enviado por <font color='blue'><b>" . $row["usuario"] . "</b></font> el " . $row["fecha"] . "</autor>";
             echo "<br/>" . $row["texto"];
             echo "<br/>";
             ?>
             <input type="submit" id="enviar2" name="enviar2" value="Borrar"/>
             <?php
            echo "</div>";
           }
         echo "</form>";
        if(isset($_POST["enviar2"])) {
            $id = $row["id"];
            $eliminar = "DELETE FROM PANEL WHERE id=$id";
            $resultado_eliminar = mysqli_query($link, $eliminar);
            header('Location: panel.php');
           }
    ?>
 

[pateketrueke]

El tema, es que antes usabas $row para obtener el ID a borrar, lo cual está mal porque para eso estás pasando el ID por el formulario.

Además, dado que usas header() debería ir el código que hace el DELETE mucho antes de comenzar a imprimir siquiera algo.

Revisa las otras respuestas que te han dado, básicamente la solución ahí está, no más debes reflexionar al respecto.

[rodrypaladin]

Me da rabia porque no os sigo apenas lo que intentáis explicarme :S valoro el esfuerzo por intentar que lo entienda, xd

Creo que entiendo que antes borraba todo porque el condicional de borrar estaba dentro del bucle. ¿ Pero ahora ? si no está dentro por que no borra nada si la sentencia sql es la misma ?

[pateketrueke]

Iniciado por rodrypaladin

Me da rabia porque no os sigo apenas lo que intentáis explicarme :S valoro el esfuerzo por intentar que lo entienda, xd

Creo que entiendo que antes borraba todo porque el condicional de borrar estaba dentro del bucle. ¿ Pero ahora ? si no está dentro por que no borra nada ?

Porque antes sólo usabas el $_POST para identificar la acción de borrar, y en el bucle usabas $row para obtener el $id en cuestión, por ende borraba todos porque no hacías ninguna distinción al respecto.

La idea de usar un formulario (o un enlace) al fin y al cabo es pasar valores entre páginas (o la misma página) de modo que los utilices.

Tu formulario solo determina una acción pero no lleva ningún identificador al respecto del registro a borrar.

[rodrypaladin]

Vale, creo que te entiendo algo mejor. Lo que ocurre ahora es que no se ponerlo en práctica, por mas que pruebe y pruebe no elimina ningún registro. ¿ Podrías ir poniendo los códigos que he puesto con lo que vas comentando y a que se va cambiando ? Se Así lo vería más claro y lo localizaría mejor

[pateketrueke]

Revisa el comentario de chivaker, básicamente esa es la solución.

[chivacker]

Te lo intentaré explicar. Puedes hacerlo en una sola página pero el flujo de la aplicación sería al revés.

Modificando tu código quedaría así:

Código:

<?php
if(!isset($_POST['id_borrar'])){
$id = $_POST['id_borrar'];
$eliminar = "DELETE FROM PANEL WHERE id='$id'";
$resultado_eliminar = mysqli_query($link, $eliminar);
}

             $query ="SELECT * FROM PANEL ORDER BY id DESC" or die("Error en la consulta. " . mysqli_error($link));
             $resultado = mysqli_query($link, $query);
             ?>
             
            <?php
             while($row = mysqli_fetch_array($resultado)) {
             echo "<div class='box-grey'>";
             echo "<autor>Enviado por <font color='blue'><b>" . $row["usuario"] . "</b></font> el " . $row["fecha"] . "</autor>";
             echo "<br/>" . $row["texto"];
             echo "<br/>";
             ?>
             <form method="POST" action="panel.php">
              <input type="hidden" id="<?php echo "$row['id']"; ?>" name="id_borrar" value="Borrar"/>
             <input type="submit" id="enviar2" name="enviar2" value="Borrar"/>
             </form>
             <?php
            echo "</div>";
           }
    ?>

[rodrypaladin]

Ahora os cuento, que tengo un problema de comillas en esta linea, se jutan 3 grupos

Código PHP:

<input type="hidden" id="<?php echo "$row['id']"; ?>" name="id_borrar" value="Borrar"/>
 

[pateketrueke]

Claro, ahí el error son las comillas de sobra:

Código PHP:

// MAL
echo "$row['id']";
 
// BIEN
echo $row['id'];
 

[chivacker]

Iniciado por rodrypaladin

Ahora os cuento, que tengo un problema de comillas en esta linea, se jutan 3 grupos

Código PHP:

<input type="hidden" id="<?php echo "$row['id']"; ?>" name="id_borrar" value="Borrar"/>
 

Pruébalo así:

Código PHP:

<input type="hidden" id="<?= $row['id']; ?>" name="id_borrar" value="Borrar"/>
 

Es la etiqueta corta para hacer un echo en PHP.

Saludos

[rodrypaladin]

Código PHP:

<?php
             if(!isset($_POST['id_borrar'])){
              $id = $_POST['id_borrar'];
              $eliminar = "DELETE FROM PANEL WHERE id='$id'";
              $resultado_eliminar = mysqli_query($link, $eliminar);
            }
 
             $query ="SELECT * FROM PANEL ORDER BY id DESC" or die("Error en la consulta. " . mysqli_error($link));
             $resultado = mysqli_query($link, $query);
             ?>
             
            <?php
             while($row = mysqli_fetch_array($resultado)) {
             echo "<div class='box-grey'>";
             echo "<autor>Enviado por <font color='blue'><b>" . $row["usuario"] . "</b></font> el " . $row["fecha"] . "</autor>";
             echo "<br/>" . $row["texto"];
             echo "<br/>";
             ?>
             <form method="POST" action="panel.php">
              <input type="hidden" id="<?= $row['id']; ?>" name="id_borrar" value="Borrar"/>
             <input type="submit" id="enviar2" name="enviar2" value="Borrar"/>
             </form>
             <?php
            echo "</div>";
           }
    ?>
 

No borra ningún registro. Algo se me debe haber pasado. Pero si que entiendo mejor lo que me intentabais explicar, pero no borra el registro

[chivacker]

Comprueba que el SQL para borrar una fila de ese ID sea correcto haciendo quitando el:

Código PHP:

if(!isset($_POST['id_borrar'])){
              $id = $_POST['id_borrar'];
              $eliminar = "DELETE FROM PANEL WHERE id='$id'";
              $resultado_eliminar = mysqli_query($link, $eliminar);
            }
 

Y poniendo:

Código PHP:

$eliminar = "DELETE FROM PANEL WHERE id='2'";
              $resultado_eliminar = mysqli_query($link, $eliminar);
 

Dónde 2 sea un ID existente

[pateketrueke]

La lógica está invertida:

Código PHP:

// MAL
!isset($_POST['id_borrar'])
 
// BIEN
isset($_POST['id_borrar'])
 

El !isset() significa "si no existe", so, está al revés.

Probablemente lo correcto era !empty() que sería "si no viene vacío", so, chivaker bugs.

[chivacker]

Iniciado por pateketrueke

La lógica está invertida:

Código PHP:

// MAL
!isset($_POST['id_borrar'])
 
// BIEN
isset($_POST['id_borrar'])
 

El !isset() significa "si no existe", so, está al revés.

Probablemente lo correcto era !empty() que sería "si no viene vacío", so, chivaker bugs.

Fe de errores. Debería ser if(isset())

[rodrypaladin]

Efectivamente era sin isset en vez de !isset porque al poner directamente el numero de id, al cargar la página se borraba ese registro automático.

Aunque al dejarlo en isset, y darle al botón de borrar registro, sigue sin borrar el registro

[rodrypaladin]

He vuelto a probar a poner el número de id directamente en vez de $id y si que borra únicamente ese registro al presionar obviamente cualquier botón pero funciona. pero si dejo $id, ya no funciona. Por ahí debe estar el problema

He probado dejandolo id=$id en vez de id='$id' pero tampoco.

Puede que haya algún error aquí, si es lo que le marca el id ?

Código PHP:

id="<?= $row['id']; ?>"
 

[pateketrueke]

Creo que chivacker te está dando más problemas que soluciones, sus código tienen bugs y ni siquiera nos explica el por qué de las cosas.

Inicialmente te mostró esto:

Código PHP:

<input name="borrar" type="hidden" value="<?= row['id'] ?>" />
 

Y es correcto, salvo que no está bien la variable.

Después colocó esto (según la corrección) pero igual está mal:

Código PHP:

<input type="hidden" id="<?= $row['id']; ?>" name="id_borrar" value="Borrar"/>
 

¿Por qué?

Porque el atributo para pasar el valor es value, y no id:

Código PHP:

<input type="hidden" value="<?= $row['id']; ?>" name="id_borrar" value="Borrar"/>
 

En fin, tampoco es cosa de copiar y pegar sin razonar lo que se lee.

Y también, tampoco se trata de aportar por aportar sin explicar razón ni causa.

[rodrypaladin]

Cierto!! Ahora funciona a la perfección, veía que el error estaba por ahí, pero me fijaba en el código php y no en el html.

Mil gracias, con esto creo que ya está completo mi panel de administrador, si por casualidad quieren ver como quedó me lo decís por privado. Gracias de nuevo

[chivacker]

Me alegro de que todo haya salido bien. Y que pateketrueke haya sabido ver los fallos que he cometido, para eso estamos en foro de ayuda =D