Checkbox con PHP

[portela85]

Buenas compañeros! a ver si me pueden ayudar con mi problema que me va a estallar la cabeza.

1. A partir de una consulta genero un listado de trabajos y que muestro en pantalla. Cada elemento está identificado por un idtrabajo y hay un checkbox por cada línea.

2. Quiero poder seleccionar varios de esos trabajos y cuando pulse el botón de validar, que se me actualicen los valores de esos trabajos.

mi problema yo creo que está en la validación del checkbox ya que siempre me lo pilla como si estuviera en estado activo (valor 1).

Os dejo el código que he hecho:

El checkbox en la plantilla:

Código:

 <td><input type="checkbox" name="validacion"/></td>

La función que valida el checkbox

Código:

    public function validarCampoValidacion($validacion){
            if ($validacion == null || $validacion == ''){
                $valido=0;
            }
            else{
                $valido=1;
            }
            return $valido;
	echo $valido;
    }

el código que recoge las variables del formulario y ejecuta la consulta:

Código:

		   if (isset($_POST['idtrabaj']) && count($_POST['idtrabaj'])>0) {
    $i=1;
    
    foreach($_POST['idtrabaj'] as $trabajo) {
	$j=1;
	
			$valid = new InformeCliente($_POST['idtrabajo']);
			$valido = new InformeCliente($_POST['idtrabajo']);
			$valido=$valid->validarCampoValidacion($_POST['validacion']);
				
				
			if ($valido=0){
				
			}
			else{
			  $query2="UPDATE trabajos SET estadocliente ='SI' WHERE idtrabajo ='$trabajo'";
			    echo $query2;
		        
			$consulta = new Consulta($query2);
		        
			if(!$consulta){throw new Exception("Error al validar las Actividades");}   
			}
			  
			}

Es decir que me hace tantos "UPDATE = SI" como trabajos hay.

Bueno muchas gracias por adelantado

[repara2]

Este echo no se ejecuta nunca:

Código PHP:

return $valido;
    echo $valido;
 

Estás dentro de un bucle foreach pero siempre pasas la misma variable a la función:

Código PHP:

foreach($_POST['idtrabaj'] as $trabajo) {
    $j=1;
    
            $valid = new InformeCliente($_POST['idtrabajo']);
            $valido = new InformeCliente($_POST['idtrabajo']);
            $valido=$valid->validarCampoValidacion($_POST['validacion']);
 

[portela85]

Muchas gracias por contestar! Pero me siguen saliendo dudas.
Cuando dices que siempre paso la misma variable, te refieres a $_POST['validacion'] cierto?

Si intento lo que viene a continuación, me sale el mensaje "Warning: Invalid argument supplied for foreach() in"

Código:

    foreach($_POST['idtrabaj'] as $trabajo) {
	$j=1;
	foreach($_POST['validacion'] as $validar){
	    $m=1;

¿cómo puedo hacer para pasarle ambas variables a la vez? Es decir el identificador del trabajo y el valor que haya en ese momento en el checkbok?

Muchas gracias de nuevo!

[Nemutagk]

sería mejor que publiques el código del formulario

[portela85]

Gracias! la plantilla del formulario sería algo así

Código:

<table border="1" bgcolor="#ffffff" cellspacing="0" cellpadding="3" width="50%" align="left">
<form action="informecliente.admin.php" method="post">
    
<input type="hidden" name="accion" value="<?php if(isset($accion)) print $accion;?>">

<?php 
if (isset($resultado_informe) && count($resultado_informe)>0) {
    $i=1;
    foreach($resultado_informe as $resultado) {
	$j=1;
	
?>

<tr>
    <td><input style="visibility:hidden; width:1px; " value="<?=$resultado['idtrabajo']?>" name="idtrabaj[<?=$resultado['idtrabajo']?>]"/></td>
    <td><?=$resultado['cliente'];?></td>
    <td><?=$resultado['proyecto'];?></td>
    <td><?=$resultado['descripcion'];?></td>
    <td><?=$resultado['actividad'];?></td>
    <td><?=$resultado['f_inicio'];?></td>
    <td><?=$resultado['talla'];?></td>
    <td><?=$resultado['cantidad'];?></td>
    <td><?=$resultado['escalado'];?></td>
    <td><?=$resultado['preciototalK25'];?></td>
    <td><?=$resultado['nocturna'];?></td>
    <td><?=$resultado['comentarios'];?></td>
    <td><?=$resultado['estadocliente'];?></td>
    <td><?=$resultado['comentariosclientes'];?></td>
    <td><?=$resultado['comentariosaltran'];?></td>
    <td><a href="informecliente.admin.php?accion=vercompleto&resultado=<?=$resultado['idtrabajo'];?>">Ver Completo</a> </td>
    <td><input type="checkbox" value"on/off" name="validacion"/></td>
    
</tr>

<tr>
<td colspan="15" align="right">
<input class="envio" type="submit" name="" value="Validar">
</td>
</tr>

</form>
</table>

[Nemutagk]

Tienes varios problemas, para comenzar el campo 'validacion' no lo estas definiendo como un array, por lo tanto estas sobre escribiendo su valor, ahora, en la validación que haces no estas comparando, estas definiendo!

Código PHP:

if ($validar = 0) { //defines el valor 0  $validar
 
if ($validar == 0) { //comparas si el valor de $validar es 0
 

por otro lado, no me parece del todo correcta la validación, puedes hacerlo directamente en el loop, eliminas lineas de código innecesarias y tendrás una mejor código...

Código PHP:

// tu código
$valid = new InformeCliente($_POST['idtrabajo']);
            $valido = new InformeCliente($_POST['idtrabajo']);
            $valido=$valid->validarCampoValidacion($_POST['validacion']);
                
                
            if ($valido=0){
                
            }
            else{
              $query2="UPDATE trabajos SET estadocliente ='SI' WHERE idtrabajo ='$trabajo'";
                echo $query2;
                
            $consulta = new Consulta($query2);
                
            if(!$consulta){throw new Exception("Error al validar las Actividades");}   
            }
 
//código "resumido"
if (isset($_POST['validar'][$item])) {
$query2="UPDATE trabajos SET estadocliente ='SI' WHERE idtrabajo ='$trabajo'";
echo $query2;
$consulta = new Consulta($query2);
if(!$consulta){throw new Exception("Error al validar las Actividades");}
}
 

Ojo, la variable $item deberás definirla en algún lado eh ir incrementandola por cada item, y obviamente debes primero convertir la variable 'validacion' en un array en el formulario

[portela85]

Muchas gracias compañeros! ya casi casi lo tengo! en cuanto esté cuelgo la solucióno. un saludo!