[SOLUCIONADO] Problemas al actualizar bd

odek · #1 (**permalink**) 04/08/2014, 10:11

Quiero actualizar mi base de datos pero no logro obtener el resultado, este es el código de la pagina que solicita los datos a actualizar.............

Código HTML:

Ver original</tr>
    
      <tr align="center">
      <td>FECHA</td>
      <td>FORMA DE PAGO</td>
      <td>REFERENCIA</td>
      <td>CONCEPTO</td>
      <td>CANTIDAD</td>
      <td>IMPORTE FINAL</td>
      <td>IMPUESTO DESGLOSADO</td>
      <td>FINALIZADO</td>
    </tr>
    
    <tr align="center">
    <form name="form1" method="post" action="act.php"><td><input type="date" name="fecha" value='.$fila['fecha'].' size="10"></td>
    <td><select name="pago" required>
          <option>'.$fila['pago'].'</option>
          <option value="Cheque">Cheque</option>
          <option value="Efectivo">Efectivo</option>
          <option value="Factura">Factura</option>
          <option value="Nota">Nota</option>
          <option value="Transferencia">Transferencia</option>
          <option value="Otro">Otro</option>
          </select>
          </td>
        <td><input type="text" name="factura" size="12" value='.$fila['factura'].'></td>
        <td><select name="concepto" required>
          <option>'.$fila['concepto'].'</option>
          <option value="Gobierno">Gobierno</option>
          <option value="Mensajeria">Mensajeria</option>
          <option value="Papeleria">Papeleria</option>
          <option value="Refacciones">Refacciones</option>
          <option value="Telefonia">Telefonia</option>
          <option value="Tecnologia">Tecnologia</option>
          <option value="Transporte">Transporte</option>
          <option value="Salud">Salud</option>
          <option value="Seguros">Seguros</option>
          <option value="Viaticos">Viaticos</option>
          </select></td>
          <td><input name="unidad" type="text" required size="10" value='.$fila['unidad'].'></td>
          <td><input type="text" name="total" size="9" value='.$fila['total'].'></td>
          <td align="center">
          <label>
            <input name="iva" type="radio" required="required" id="iva_0" value='.$fila['total'].'>
            No cambiar, me quedo como estaba<br></label>
          <label>
            <input name="iva" type="radio" required="required" id="iva_0" value="16">
            Desglosar impuesto del 16</label><br>
          <label>
            <input name="iva" type="radio" required="required" id="iva_1" value="0">
            Sin impuesto</label>
           <label>
          <td align="center"> <input name="pendiente" type="radio" required="required" id="pendiente_0" value="Pendiente">
            Faltan cosas<br>
          </label>
          <label>
            <input name="pendiente" type="radio" required="required" id="pendiente_1" value="-">
           He finalizado</label></td>
    </tr>
    
    <tr>
    <td colspan="7" align="center">OBSERVACIONES</td>
    <td rowspan="7"><input type="submit"></td>
    </tr>
    <tr>
    <td colspan="7" align="center"><input type="textarea" name="observaciones" size="135" value="'.$fila['observaciones'].'"></td>
    </tr></form>
    </table>

esta es la pagina que proceda los datos para actualizar

Código PHP:

Ver original$usuario = isset($_SESSION['usuario']) ?$_SESSION['usuario'] : null;
$contrasena = isset($_SESSION['contrasena']) ?$_SESSION['contrasena'] : null;
 
if (isset($_POST['fecha'])) {
$fecha = $_POST['fecha'];
} else {
$fecha = "";    
}
if (isset($_POST['pago'])) {
$pago = $_POST['pago'];
} else {
$pago = ""; 
} 
if (isset($_POST['factura'])) {
$factura = $_POST['factura'];
} else {
$factura = "";  
} 
if (isset($_POST['unidad'])) {
$unidad = $_POST['unidad'];
} else {
$unidad = "";   
} 
if (isset($_POST['subtotal'])) {
$subtotal = $_POST['subtotal'];
} else {
$subtotal = ""; 
} 
if (isset($_POST['iva'])) {
$iva = $_POST['iva'];
} else {
$iva = "";  
} 
if (isset($_POST['total'])) {
$total = $_POST['total'];
} else {
$total = "";    
} 
if (isset($_POST['observaciones'])) {
$observaciones = $_POST['observaciones'];
} else {
$observaciones = "";    
} 
if (isset($_POST['pendiente'])) {
$pendiente = $_POST['pendiente'];
} else {
$pendiente = "";    
} 
if (isset($_SESSION['fecha'])) {
$antiguo = $_SESSION['fecha'];
} else {
$antiguo = "";  
}
 
$sentencia = "UPDATE IO018TOQ059N62014 SET fecha='".$fecha."', pago='".$pago."', factura='".$factura."', subtotal='".$subtotal."', iva='".$iva."', total='".$total."', observaciones='".$observaciones."', pendiente='".$pendiente."' WHERE fecha='".$antiguo."'
";  
 
$resultado = mysqli_query($conexion,$sentencia);
 
mysqli_close($conexion);

Esto me enseñe con mysql pero estoy pasando todo a mysqli pero no se en que estoy mal, espero puedan ayudarme. gracias.

Triby · #2 (**permalink**) 04/08/2014, 12:58

Antes de averiguar el problema, unos cuantos comentarios:

1- No es recomendable tener un echo con toda esa cantidad de información, porque te es muy ilegible el código y complicarás mucho en caso de necesitar modificaciones; en todo caso, te recomiendo la sintáxis HEREDOC

2- Creo que en todos los inputs estás omitiendo las comillas para el valor, ejemplo:
<input type="date" name="fecha" value='.$fila['fecha'].' size="10">
<input type="date" name="fecha" value="'.$fila['fecha'].'" size="10">
Casi estoy seguro que la causa es lo mencionado en el punto 1

3- Puedes simplificar los if's donde tomas los valores de $_POST con asignación ternaria, es lo mismo, pero "más barato":

Código PHP:

Ver original// Lo que tú haces:
if (isset($_POST['fecha'])) {
    $fecha = $_POST['fecha'];
} else {
    $fecha = "";   
}
 
// Con asignación ternaria:
$fecha = (isset($_POST['fecha'])) ? $_POST['fecha'] : '';

4- Estás tomando todos los campos tal como vienen del formulario, sin escaparlos; aparte de ser la causa más común de errores, también implica riesgos de seguridad por inyección SQL; Lee esto: http://www.forosdelweb.com/f18/aport...8/#post4265377

Bueno, ahora sí, con tu "problema real"... cuál es el error que obtienes?

odek · #3 (**permalink**) 04/08/2014, 21:42

si quito where funciona todo el codigo, pero actualiza todos los campos a la vez con la misma información pero al agregar el where no funciona.

Ayudenme a encontrar el error.

Gracias

CODIGO

if (isset($_POST['id'])) {
$id = $_POST['id'];
} else {
$id = "";
}
if (isset($_POST['fecha'])) {
$fecha = $_POST['fecha'];
} else {
$fecha = "";
}
if (isset($_POST['pago'])) {
$pago = $_POST['pago'];
} else {
$pago = "";
}

$sentencia = "UPDATE IO018TOQ059N62014 SET fecha='".$fecha."', pago='".$pago."' WHERE id='".$id."'
";

$resultado = mysqli_query($conexion,$sentencia);

mysqli_close($conexion);

echo '
<html>
<head>
<meta http-equiv="REFRESH" content="0;url=pruebaap.php">
</head>
</html>
';

pateketrueke · #4 (**permalink**) 04/08/2014, 21:52

No hay forma de que podamos adivinar como se ve la consulta final con todo y WHERE, ¿o esperas a que de verdad adivinemos?

Digo, leer código no nos dice nada, así que si no depuras la consulta nosotros tampoco lo haremos desde acá.

Triby · #5 (**permalink**) 04/08/2014, 22:27

@odek, es contra las políticas de FDW iniciar dos o más temas sobre el mismo problema; por favor continúa aquí tu consulta y responde a las preguntas que se te han hecho, de otra forma será más difícil que obtengas ayuda.

odek · #6 (**permalink**) 05/08/2014, 10:11

PROBLEMAS CON WHERE.....

Aqui esta el codigo que contienes los datos

<?php
$sentencia = "SELECT * FROM IO018TOQ059N62014 ORDER by fecha";
$resultado = mysqli_query($conexion,$sentencia);
while ($fila = mysqli_fetch_array($resultado)){
echo "

<td>".$fila['fecha']."</td>
<td>".$fila['pago']."</td>

<td bgcolor='#FF7777'><a href='EIO018TOQ059N62014.php?fecha=".$fila['fecha']."&pago=".$fila['pago']."'>Eliminar</a><br>
<a href='AIO018TOQ059N62014.php?fecha=".$fila['fecha']."&pago=".$fila['pago']."'>Actualizar</a></td>
</tr>";
}
?>

Aqui proceso los datos para ser actualizados

<?php
echo '
<tr align="center">
<form name="form1" method="post" action="act.php"><td><input type="date" name="fecha" value="'.$fila['fecha'].'" size="10"></td>
<td><select name="pago" required>
<option>"'.$fila['pago'].'"</option>
<option value="Cheque">Cheque</option>
<option value="Efectivo">Efectivo</option>
<option value="Factura">Factura</option>
<option value="Nota">Nota</option>
<option value="Transferencia">Transferencia</option>
<option value="Otro">Otro</option>
</select>
</td>
</tr>
</form>
</table>
';
}
?>

Hasta aqui obtengo todos los datos, ahora al presionar un botón de enviar llamo a una pagina para que los procese y actualize

<?php
$usuario = isset($_SESSION['usuario']) ?$_SESSION['usuario'] : null;
$contrasena = isset($_SESSION['contrasena']) ?$_SESSION['contrasena'] : null;

if (isset($_POST['id'])) {
$id = $_POST['id'];
} else {
$id = "";
}
if (isset($_POST['fecha'])) {
$fecha = $_POST['fecha'];
} else {
$fecha = "";
}
if (isset($_POST['pago'])) {
$pago = $_POST['pago'];
} else {
$pago = "";
}

$sentencia = "UPDATE IO018TOQ059N62014 SET fecha='".$fecha."', pago='".$pago."' WHERE id='".$id."'
";

$resultado = mysqli_query($conexion,$sentencia);

mysqli_close($conexion);

echo '
<html>
<head>
<meta http-equiv="REFRESH" content="0;url=pruebaap.php">
</head>
</html>
';

?>

Aqui tengo el problema, al agregar where no logro procesar nada, en cambio si lo elimino efectivamente cambia los datos pero de todas las columnas y no de una sola.

Mi pregunta es ¿que parte de la condición WHERE esta mal?

gnzsoloyo · #7 (**permalink**) 05/08/2014, 10:16

Cita:

Aqui tengo el problema, al agregar where no logro procesar nada, en cambio si lo elimino efectivamente cambia los datos pero de todas las columnas y no de una sola.

La única forma en que esto:

Código PHP:

Ver original$sentencia = "UPDATE IO018TOQ059N62014 SET fecha='".$fecha."', pago='".$pago."' WHERE id='".$id."'";

genere ese problema (no actualice), es si y sólo si la variable $id viene nula, vacía o no contiene ningún ID valido para la tabla.
Al menos haz esto, y muestranos qué queda armado:

Código PHP:

Ver original$sentencia = "UPDATE IO018TOQ059N62014 SET fecha='".$fecha."', pago='".$pago."' WHERE id='".$id."'"; 
// Mostrando contenido de la query ya armada
echo $sentencia ;

Posdata: Haznos el favor de usar el Highlight "PHP para poner el codigo. Permitirá que podamos entender mejor lo que estás posteando.
Para eso están.

odek · #8 (**permalink**) 05/08/2014, 10:24

este es el resultado del echo $sentencia.......

Código PHP:

Ver original<?php
session_start();
        
$usuario = isset($_SESSION['usuario']) ?$_SESSION['usuario'] : null;
$contrasena = isset($_SESSION['contrasena']) ?$_SESSION['contrasena'] : null;
 
if (isset($_POST['id'])) {
$id = $_POST['id'];
} else {
$id = "";   
}
if (isset($_POST['fecha'])) {
$fecha = $_POST['fecha'];
} else {
$fecha = "";    
}
if (isset($_POST['pago'])) {
$pago = $_POST['pago'];
} else {
$pago = ""; 
} 
 
 
$sentencia = "UPDATE IO018TOQ059N62014 SET fecha='".$fecha."', pago='".$pago."' WHERE id='".$id."'
";  
 
$resultado = mysqli_query($conexion,$sentencia);
 
echo "$sentencia";
 
mysqli_close($conexion);
 
?>

UPDATE IO018TOQ059N62014 SET fecha='2014-07-30', pago='""Cheque""' WHERE id=''

Efectivamente el id no tiene ningún valor, mi error es que no lo solicite antes o que es lo que pasa????

gnzsoloyo · #9 (**permalink**) 05/08/2014, 10:29

Estás verificando que el parámetro este definido, pero no que contenga algo... Y no tiene sentido hacer el UPDATE si al menos UNA de las variables no contiene datos...

Código PHP:

Ver original<?php
$usuario = isset($_SESSION['usuario']) ?$_SESSION['usuario'] : null;
$contrasena = isset($_SESSION['contrasena']) ?$_SESSION['contrasena'] : null;
 
if (isset($_POST['id']) && $_POST['id'] != '' && isset($_POST['pago']) && $_POST['pago'] != '' && isset($_POST['fecha']) && $_POST['fecha'] != '')
{
$id = $_POST['id'];
$fecha = $_POST['fecha'];
$pago = $_POST['pago'];
 
$sentencia = "UPDATE IO018TOQ059N62014 SET fecha='".$fecha."', pago='".$pago."' WHERE id='".$id."'
";
 
$resultado = mysqli_query($conexion,$sentencia);
 
mysqli_close($conexion);
}
echo '
<html>
<head>
<meta http-equiv="REFRESH" content="0;url=pruebaap.php">
</head>
</html>
';
 
?>

¿Me puedes decir qué sentido tiene hacer ese ELSE:

Código PHP:

Ver originalif (isset($_POST['id'])) {
$id = $_POST['id'];
} else {
$id = "";  
}

Tu mismo estás poniendo NADA en esa variable...
Obviamente el dato no está llegando desde ninguna parte, pero si no lo hace, entonces no debes hacer ninguna acción.

odek · #10 (**permalink**) 05/08/2014, 10:56

si $id se verifica y no lo toma WHERE, quiero entender que debo usarlo con alguna variable que contenga datos ejemplo $fecha, aun a pesar de que $id tiene un valor de autoincremento?

Pero aun así el UPDATE WHERE no cambia ningún parámetro utilizando otras variables.

gnzsoloyo · #11 (**permalink**) 05/08/2014, 11:02

Me parece que no estás entendiendo la idea...
El WHERE determina a qué registro debe realizarse la modificación. Si no aplicas ninguna condición en él, modificara todos los registros de la tabla, sin distinciones, poniendo los mismos datos en todos ellos.
En tu caso, ese "ID" debería llegar desde otra aprte, es decir, desde donde se está invocando al script, por $_POST. Si no está llegando, no puedes hacer ninguna acción, no debes hacer ninguna, y sólo devovler un mensaje de error diciendo que nose ha seleccionado el ID a modificar.
¿Se entiende?
Lo que debes verificar ahora es por qué no está llegando ese dato, pero eso no lo sabrás en ese segmento, siono en el que lo invoca.

Ahora bien, dicho eso, ¿dónde está ese "id" en este código?:

Código HTML:

Ver original<?php
echo '
<tr align="center">
<form name="form1" method="post" action="act.php"><td><input type="date" name="fecha" value="'.$fila['fecha'].'" size="10"></td>
<td><select name="pago" required>
<option>"'.$fila['pago'].'"</option>
<option value="Cheque">Cheque</option>
<option value="Efectivo">Efectivo</option>
<option value="Factura">Factura</option>
<option value="Nota">Nota</option>
<option value="Transferencia">Transferencia</option>
<option value="Otro">Otro</option>
</select>
</td>
</tr>
</form>
</table>
';
}
?>

Yo no lo veo...

odek · #12 (**permalink**) 05/08/2014, 11:21

Modifique la variable $fecha por $id para comprobar que llama ese dato.

Código PHP:

Ver original<?php
$sentencia = "SELECT * FROM IO018TOQ059N62014 ORDER by fecha";
$resultado = mysqli_query($conexion,$sentencia);
while ($fila = mysqli_fetch_array($resultado)){
echo "
<input type='hidden' name='id' value='".$fila['id']."'>
<td>".$fila['fecha']."</td>
<td>".$fila['pago']."</td>
 
<td bgcolor='#FF7777'><a href='EIO018TOQ059N62014.php?fecha=".$fila['fecha']."&pago=".$fila['pago']."'>Eliminar</a><br>
<a href='AIO018TOQ059N62014.php?fecha=".$fila['fecha']."&pago=".$fila['pago']."'>Actualizar</a></td>
</tr>";
}
?>

Realizo la misma modificación en la pagina donde llame a los datos que serán modificados.

Código PHP:

Ver original<?php
echo '
<tr align="center">
<form name="form1" method="post" action="act.php">
<input type="hidden" name="id" value="'.$fila['id'].'"><td><input type="date" name="fecha" value="'.$fila['fecha'].'" size="10"></td>
<td><select name="pago" required>
<option>"'.$fila['pago'].'"</option>
<option value="Cheque">Cheque</option>
<option value="Efectivo">Efectivo</option>
<option value="Factura">Factura</option>
<option value="Nota">Nota</option>
<option value="Transferencia">Transferencia</option>
<option value="Otro">Otro</option>
</select>
</td>
</tr>
</form>
</table>
';
}
?>

Hasta aqui el código es correcto.

*******Pero al realizar UPDATE

Código PHP:

Ver original<?php
        
if (isset($_POST['id']) && $_POST['id'] != '' && isset($_POST['pago']) && $_POST['pago'] != '')
{
$id = $_POST['id'];
$pago = $_POST['pago'];
 
$sentencia = "UPDATE IO018TOQ059N62014 SET pago='".$pago."' WHERE id='".$id."'
";
 
$resultado = mysqli_query($conexion,$sentencia);
 
echo "$sentencia";
 
mysqli_close($conexion);
}
 
    ?>

Ya obtengo el valor de id que es 12, pero no modifica el campo porque?

odek · #13 (**permalink**) 05/08/2014, 15:19

Solucion actualizar php.........