Problemas codigo al hacer un delete

sentoki79 · #1 (**permalink**) 03/02/2016, 11:21

Buenas a todos,

estoy haciendo un backend y tengo un problema al tratar de borrar las filas de una tabla.

Me ocurre que despues de elegir con un checkbox que filas borrar, no llega a borrarlas.

Veo que el problema es que no recoge cuantas filas se han clickado. Porque he realizado un alert despues de recoger con isset si se han pulsado y siempre indica que 0 filas.

Pego el codigo:

if(isset($_POST["Id"])) {
$delete = $_POST["Id"];
$cantidad = count($delete);
for ($i=0; $i<$cantidad; $i++) {
$del_id = $delete[$i];
mysqli_query($conexion,"DELETE FROM usuarios WHERE Id='".$del_id."'")or
die("Problemas en el select:".mysqli_error($conexion));
}
echo "<script>alert('Elementos eliminado');</script>";

echo "<script language='JavaScript'>window.self.location='BBDD_u suarios.php';</script> ";
}

echo"
<form method='post' action='form2.php'>
<input type='submit' name='borrar' value='Borrar fila'>
</form>
";

Y pego el codido del checkbox:

<tr>

<td colspan='1'> <input type='checkbox' name='Id[]' value=".$Id."><span name='id'>".$reg['Id']."</span></td>

Gracias por la ayuda!

pateketrueke · #2 (**permalink**) 03/02/2016, 11:35

¿Los checkbox están dentro o fuera del formulario?

sentoki79 · #3 (**permalink**) 03/02/2016, 11:56

Estan dentro :)

pateketrueke · #4 (**permalink**) 03/02/2016, 12:45

Cita:

Iniciado por sentoki79

Estan dentro :)

¿Podrías mostrar el código completo?

Porque para que digas que nunca vienen dichos datos entonces algo anda mal con dicho formulario.

sentoki79 · #5 (**permalink**) 03/02/2016, 13:00

Gracias por tratar de echarme una mano compañero, incluyo el codigo:

<html>

<head>
<title>Usuarios</title>
<META name='robot' content='noindex, nofollow'>
</head>

<?php

$conexion=mysqli_connect("localhost","root","","us uarios")or die("Problemas con la conexión");
$registros=mysqli_query($conexion,"select * from usuarios")or
die("Problemas en el SELECT:".mysqli_error($conexion));
while($reg=mysqli_fetch_array($registros))
{
$Id=$reg['Id'];

echo "
<body>

<div align='center'>
<table border='0' width='600' style='font-family: Verdana; font-size: 8pt' id='table1'>

<form method='POST' action='form2.php'>
<tr>
<td colspan='1'><p align='center'><b>Id: </b></td>
<td width='50%'><p align='center'><b>nombre usuario:</b></td>
<td width='50%'><p align='center'><b>contrasenya:</b></td>
<td width='50%'><p align='center'><b>nombre :</b></td>
<td width='50%'><p align='center'><b>primer apellido:</b></td>
<td width='50%'><p align='center'><b>segundo apellido:</b></td>
<td width='50%'><p align='center'><b>DNI:</b></td>
<td width='50%'><p align='center'><b>Email:</b></td>
<td width='50%'><p align='center'><b>Telefono:</b></td>
<td width='50%'><p align='center'><b>Privilegios:</b></td>
</tr>
<tr>

<td colspan='1'> <input type='checkbox' name='Id[]' value=".$Id."><span name='id'>".$reg['Id']."</span></td>

<td width='50%'><p align='center'><input type='text' name='user' size='10' value='".$reg['NombreUsuario']."'></td>

<td width='50%'><p align='center'><input type='text' name='password' size='10' value='".$reg['Password']."'>

<td width='50%'><p align='center'><input type='text' name='nombre' size='10' value='".$reg['Nombre']."'></td>

<td width='50%'><p align='center'><input type='text' name='apellido1' size='10' value='".$reg['Apellido1']."'>

<td width='50%'><p align='center'><input type='text' name='apellido2' size='10' value='".$reg['Apellido2']."'>

<td width='50%'><p align='center'><input type='text' name='DNI' size='10' value='".$reg['DNI']."'>

<td width='50%'><p align='center'><input type='text' name='email' size='10' value='".$reg['Email']."'>

<td width='50%'><p align='center'><input type='text' name='telefono' size='10' value='".$reg['Telefono']."'>

<td width='50%'><p align='center'><input type='text' name='privilegio' size='10' value='".$reg['Privilegio']."'>

<tr>
<td width='50%'> </td>
<td width='50%'> </td>
</tr>
<input type='hidden' name='id' value='".$reg['Id']."'>
<tr>
<td width='100%' colspan='2'>
<p align='center'>
<input type='submit' value='Actualizar datos' name='B1'></td>
</tr>
</form>
</table>
<tr/>
</div>

";
}
if(isset($_POST["Id"])) {
$delete = $_POST["Id"];
$cantidad = count($delete);
for ($i=0; $i<$cantidad; $i++) {
$del_id = $delete[$i];
mysqli_query($conexion,"DELETE FROM usuarios WHERE Id='".$del_id."'")or
die("Problemas en el DELETE:".mysqli_error($conexion));
}

echo "<script>alert('Elementos eliminado');</script>";

echo "<script language='JavaScript'>window.self.location='BBDD_u suarios.php';</script> ";
}

echo"
<form method='post' action='form2.php'>
<input type='submit' name='borrar' value='Borrar fila'>
</form>

<form method='post' action='Nuevo_Usuario.php'>
<input type='submit' name='nuevo' value='Introducir nuevo'>
</form>
";
mysqli_close($conexion);
?>

</body>

</html>

pateketrueke · #6 (**permalink**) 03/02/2016, 13:03

Si estás usando un while() para imprimir un formulario por cada registro entonces no hay necesidad de usar la notación de array, ese sería un error a considerar.

Ahora, cuando recibas el formulario (una vez corregido) ¿podrías usar var_dump() para revisar qué valores vienen en el POST?

Código PHP:

Ver originalvar_dump($_POST);

¿Viene el Id o no?

sentoki79 · #7 (**permalink**) 03/02/2016, 13:48

La verdad es que no lo pinta por pantalla.
He pegado el var_dump justo aqui:

var_dump($_POST);
if(isset($_POST["Id"])) {
$delete = $_POST["Id"];
$cantidad = count($delete);
for ($i=0; $i<$cantidad; $i++) {
$del_id = $delete[$i];
mysqli_query($conexion,"DELETE FROM usuarios WHERE Id='".$del_id."'")or
die("Problemas en el DELETE:".mysqli_error($conexion));
}

Porque si lo pongo dentro del isset esperando a que entre dentro del if, no se ejecuta var_dump ya que no recoge por post y no entra dentro del if.

Te pongo el resultado de var_dump fuera del isset:

array(1) { ["borrar"]=> string(11) "Borrar fila" }

"borrar" es el identificador del submit.

Y como has indicado he eliminado el array del identificador del checkbox.

Saludos

chronos682 · #8 (**permalink**) 04/02/2016, 08:18

El código que pegaste arriba es de la página form2.php? Porque hacia allá es a donde estás dirigiendo el formulario. Si no tienes una página form2.php borra eso del action del formulario y déjalo así: action="".

sentoki79 · #9 (**permalink**) 04/02/2016, 09:09

Buenas,

he probado a hacer lo que me indicas, pero no ha funcionado
Sigue indicando que la variable Id esta vacia ...debo hacer un if (isset($_POST['Id'])) dentro del isset del boton borrar? o directamente haciendo un post al identificado del checkbox ya recoje los checkbox que se han clickado?

chronos682 · #10 (**permalink**) 04/02/2016, 09:16

Ya veo que estás haciendo mal, tienes los botones de eliminar y de nuevo usuario en forms separados a donde estan los checkbox por eso nunca llega el resto de controles. Debes colocar el botón eliminar dentro del primer formulario y eliminar lo del action como te dije antes.

sentoki79 · #11 (**permalink**) 04/02/2016, 16:47

Muchas gracias era eso!

como indicabas esta fuera del <form> donde estaba el checkbox.

Ahora me estoy peleando para que no salga un boton de borrar por cada fila, porque sino a la hora del delete solo borra la fila del boton pulsado. Ya me imagino que es poner el boton fuera del alcance del while ...si lo consigo subo el codigo por si hay alguien interesado.

Saludos y gracias!

pateketrueke · #12 (**permalink**) 04/02/2016, 16:54

Cita:

Muchas gracias era eso!

Yo ya te había mencionado lo mismo desde un inicio, en fin...

chronos682 · #13 (**permalink**) 04/02/2016, 16:56

Cita:

Iniciado por sentoki79

Muchas gracias era eso!

como indicabas esta fuera del <form> donde estaba el checkbox.

Ahora me estoy peleando para que no salga un boton de borrar por cada fila, porque sino a la hora del delete solo borra la fila del boton pulsado. Ya me imagino que es poner el boton fuera del alcance del while ...si lo consigo subo el codigo por si hay alguien interesado.

Saludos y gracias!

Coloca el botón fuera del while para que sólo se vea al final.

sentoki79 · #14 (**permalink**) 05/02/2016, 09:44

Entendi al input del checkbox, no al boton de borrar ..no obstante muchas gracias por los consejos me han ido orientando por donde debía de ir

sentoki79 · #15 (**permalink**) 05/02/2016, 09:45

Cita:

Iniciado por pateketrueke

Yo ya te había mencionado lo mismo desde un inicio, en fin...

Entendi al input del checkbox, no al boton de borrar ..no obstante muchas gracias por los consejos me han ido orientando por donde debía de ir

pateketrueke · #16 (**permalink**) 05/02/2016, 09:53

Cita:

Iniciado por sentoki79

Entendi al input del checkbox, no al boton de borrar ..no obstante muchas gracias por los consejos me han ido orientando por donde debía de ir

Sí claro, pero es obvio que no pueden estar en formularios diferentes si tienen relación entre si.

Eso lo aprendes leyendo un manual básico de HTML sobre formularios.