[SOLUCIONADO] Problema con Update de Registro

[galaktico]

Buenos días, estoy tratando de realizar un update de registros utilizando 1 sólo archivo.

Para ello primero envío el ID por GET de la siguiente manera:

Código PHP:

<a href="modifica.php?id_producto=<?php echo $fila['id_producto']; ?>">Modifica</a>
 

Luego recibo el ID en el archivo modifica.php:

Código PHP:

<?php 
include("conexion.php");
$id = $_GET['id_producto'];
 
if(isset($_POST["enviar"])){
  $consulta = "UPDATE productos SET nombre='$nombre', categoria='$categoria' WHERE id_producto=$id";
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error());
  echo "Los datos fueron actualizados con éxito!.\n"; 
} else {
  $categorias = mysql_query ("SELECT * from categorias ORDER BY id_categoria ASC;", $conexion);
  $consulta = "SELECT * FROM productos WHERE id_producto=$id";
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error());
  $fila=mysql_fetch_array ($resultado);
?>
<form action="modifica.php" method="post">
  <input name="nombre" type="text" value="<?php echo $fila["nombre"]; ?>">
  <select name="categoria">
    <option><?php echo $fila["categoria"]; ?></option>
    <?php
        while ($fila=mysql_fetch_array($categorias))
        {
            echo "<option value='".$fila ["id_categoria"]."'>".$fila ["categoria"]."</option>";
        }
        mysql_close($conexion);
    ?>
  </select>
  <input type="submit" value="Guardar Cambios" name="enviar">
</form>
<?php 
}    
?>
 

Vengo bastante bien, pero me pasan 2 cosas:

1) Me tira el siguiente error:
You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

2) Necesito que el option del select me muestre el nombre de la categoría, ahora me levanta el ID.

Me podrán ayudar?

Muchas gracias!

[Middrel]

Te está llegando un dato vacío.

Verifica con

Código PHP:

  echo "UPDATE productos SET nombre='$nombre', categoria='$categoria' WHERE id_producto=$id";  
   

para que veas que está haciendo la consulta.

con

Código PHP:

  var_dump($_POST);  
   

para que veas que estás enviando a la consulta.

[galaktico]

Para no marear con el código saqué lo de categorías y dejé solamente esto:

Código PHP:

<?php 
include("conexion.php");
$id = $_GET['id_producto'];
 
if(isset($_POST["enviar"])){
  $consulta = "UPDATE productos SET nombre='$nombre' WHERE id_producto=$id";
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error());
  echo "Los datos fueron actualizados con éxito!.\n"; 
} else {
  $consulta = "SELECT * FROM productos WHERE id_producto=$id";
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error());
  $fila=mysql_fetch_array ($resultado);
?>
<form action="modifica.php" method="post">
  <input name="nombre" type="text" value="<?php echo $fila["nombre"]; ?>">
  <input type="submit" value="Guardar Cambios" name="enviar">
</form>
<?php 
}    
?>
 

Me levanta ok el nombre pero cuando presiono enviar me tira el error:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

No estoy pudiendo ver donde está el error, me podrán ayudar?

Muchas gracias!

[Alexis88]

Al momento de realizar la actualización, las variables $nombre y $categoria no tienen valores, o al menos no muestras en qué parte se los asignas y quizá los campos que intentas actualizar no pueden tener valores nulos. Corrige eso.

Saludos

[galaktico]

Alexis, le agregué este codigo para no enviar vacíos los campos:

Código PHP:

$categoria = $_POST['categoria'];
  $nombre = $_POST['nombre'];
 

Pero me sigue dando el mismo error.

Me podrás orientar?

Muchas gracias!

[Middrel]

El problema no es ese Galaktico.

El input que envía el nombre,

Código PHP:

 <input name="nombre" type="text" value="<?php echo $fila["nombre"]; ?>">  

recibe el valor de la consulta,

Código PHP:

  } else { 
  $consulta = "SELECT * FROM productos WHERE id_producto=$id"; 
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error()); 
  $fila=mysql_fetch_array ($resultado); 
?>  
  

entonces, cuando le das en "enviar", entra aquí:

Código PHP:

  if(isset($_POST["enviar"])){ 
  $consulta = "UPDATE productos SET nombre='$nombre' WHERE id_producto=$id"; 
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error()); 
  echo "Los datos fueron actualizados con éxito!.\n";  
   

sin nunca haber obtenido el valor de "nombre".

nombre siempre viaja vacío por lo que veo. Deberías sacar esa consulta del if.

Código PHP:

  <?php  
include("conexion.php"); 
 
$consulta = "SELECT * FROM productos WHERE id_producto=$id"; 
$resultado = mysql_query($consulta, $conexion) or die(mysql_error()); 
$fila=mysql_fetch_array ($resultado); 
 
 
$id = $_GET['id_producto']; 
$categoria = $_POST['categoria']; 
$nombre = $_POST['nombre']; 
  
if(isset($_POST["enviar"])){ 
  $consulta = "UPDATE productos SET nombre='$nombre' WHERE id_producto=$id"; 
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error()); 
  echo "Los datos fueron actualizados con éxito!.\n";  
}  
?> 
 
<form action="modifica.php" method="post"> 
     <input name="nombre" type="text" value="<?php echo $fila["nombre"]; ?>"> 
     <input type="submit" value="Guardar Cambios" name="enviar"> 
</form>  

[galaktico]

Probé de la siguiente manera:

Código PHP:

<?php 
include("conexion.php");
$id = $_GET['id_producto'];
$nombre = $_POST['nombre'];
$consulta = "SELECT * FROM productos WHERE id_producto=$id";
$resultado = mysql_query($consulta, $conexion) or die(mysql_error());
$fila=mysql_fetch_array ($resultado);
 
if(isset($_POST["enviar"])){
  $consulta = "UPDATE productos SET nombre='$nombre' WHERE id_producto=$id";
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error());
  echo "Los datos fueron actualizados con éxito!.\n"; 
} else {
?>
<form action="modifica.php" method="post">
  <input name="nombre" type="text" value="<?php echo $fila["nombre"]; ?>">
  <input type="submit" value="Guardar Cambios" name="enviar">
</form>
<?php 
}    
?>
 

Pero me sigue dando el error:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

Se les ocurre alguna solución?

Muchas gracias!

[Drowna]

Ahm no quiero parecer tonta, pero, cuando realizas la segunda consulta, es decir cuando pulsas enviar de donde sacas el valor de la variable $id?. Porque ya no la estas enviando por GET. Te propongo una cosa a ver si es lo que sospecho.

En el formulario añade un input de tipo hidden y que tenga como value el valor de la variable $id y en el if extrae también el valor de este campo. sería algo asi.

Código PHP:

<?php 
include("conexion.php");
$id = $_GET['id_producto'];
$nombre = $_POST['nombre'];
$consulta = "SELECT * FROM productos WHERE id_producto=$id_r";
$resultado = mysql_query($consulta, $conexion) or die(mysql_error());
$fila=mysql_fetch_array ($resultado);
 
if(isset($_POST["enviar"])){
$id_producto = $_POST['id']
  $consulta = "UPDATE productos SET nombre='$nombre' WHERE id_producto=$id_producto";
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error());
  echo "Los datos fueron actualizados con éxito!.\n"; 
} else {
?>
<form action="modifica.php" method="post">
  <input name="nombre" type="text" value="<?php echo $fila["nombre"]; ?>">
  <input type="submit" value="Guardar Cambios" name="enviar">
<input type="hidden" name="id" value="<?php echo $id ?>">
</form>
<?php 
}    
?>
 

[bookmaster]

Ya nada, he puesto precisamente lo mismo que Drowna, pero el le dio a enviar antes.

[galaktico]

Yo lo planteé así:

1) Utilizo un buscador para buscar el producto a modificar.
2) En la página resultado listo los resultados y de ahí voy al archivo modifca.php pasando el ID por GET de la siguiente manera:

Código PHP:

<a href="modifica.php?id_producto=<?php echo $fila['id_producto']; ?>">Modifica</a>
 

Lo que necesito es hacer un Update en la base trayendo el ID de otro archivo.

De ahí es de donde viene el valor de $id, por eso lo rescato así:

Código PHP:

$id = $_GET['id_producto'];
 

Y luego intento realizar todo en el mismo archivo:

Código PHP:

<?php 
include("conexion.php");
$id = $_GET['id_producto'];
 
if(isset($_POST["enviar"])){
  $consulta = "UPDATE productos SET nombre='$nombre' WHERE id_producto=$id";
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error());
  echo "Los datos fueron actualizados con éxito!.\n"; 
} else {
  $consulta = "SELECT * FROM productos WHERE id_producto=$id";
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error());
  $fila=mysql_fetch_array ($resultado);
?>
<form action="<?=$_SERVER['PHP_SELF']?>" method="post">
  <input name="nombre" type="text" value="<?php echo $fila["nombre"]; ?>">
  <input type="submit" value="Guardar Cambios" name="enviar">
</form>
<?php 
}    
?>
 

Lo estoy planteando mal?

[Drowna]

En parte si, porque cuando pulsas el botón enviar le indicas al script que te lea de nuevo todo el script de la página pero no le estas diciendo que obtenga ese valor de nuevo por get si no por post por lo que el $id te lo enviará como una cadena vacia

[galaktico]

Entonces hay algo que no entiendo conceptualmente.

Si parto de un buscador para elegir el producto a modificar, indefectiblemente tengo que pasar un ID.

Luego modifico el Form con los nuevos valores y luego indefectiblemente tengo que volver a hacer Post o Get, por lo tanto pierdo el ID de la página resultado anterior?

Entonces como es que se debe realizar correctamente?

Perdón por la consulta, pero tengo realizado el ABM casi completo!

Me falta solamente el modifica.php

Si partira de la página resultado, envío el ID por GET.

Código PHP:

<a href="elimina.php?id_producto=<?php echo $fila['id_producto']; ?>">Modifica</a>
 

Luego en el archivo modifica.php tengo mi formulario:

Código HTML:

<form action="" method="post">
  <input name="nombre" type="text">
</form>
 

De qué manera conceptualmente debería hacer el Update?

[gnzsoloyo]

Una query armada en PHP con esta sintaxis:

Código SQL:

UPDATE productos SET nombre='$nombre' WHERE id_producto=$id
 

O esta:

Código SQL:

SELECT * FROM productos WHERE id_producto=$id
 

Puede producir un error de sintaxis si y SOLO SI la variable $id es nula.
Únicamente.

Vale decir, a pesar de lo que insistas, el ID no está llegando a la query, y no veo que lo estés validando... al menos no lo estás validando bien.

[galaktico]

Yo parto de un buscador y luego de la página resultado paso el ID a modificar por GET, luego el formulario me levanta los datos cargados en la base y allí los modifico, pero después que? si lo hago con un archivo por separado pierdo el ID y si lo hago en el mismo archivo lo tengo vacío al ID.

Me parece que sigo sin entenderlo conceptualmente.

[galaktico]

Siguiendo la lógica que me plantearon en todo el post debería funcionar así:

Código PHP:

<?php 
include("conexion.php");
 
if(isset($_POST["enviar"])){
  $id = $_GET['id_producto'];
  $consulta = "UPDATE productos SET nombre='$nombre' WHERE id_producto=$id";
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error());
  echo "Los datos fueron actualizados con éxito!.\n"; 
} else {
  $id = $_GET['id_producto'];
  $consulta = "SELECT * FROM productos WHERE id_producto=$id";
  $resultado = mysql_query($consulta, $conexion) or die(mysql_error());
  $fila=mysql_fetch_array ($resultado);
?>
<form action="modifica.php" method="post">
  <input name="nombre" type="text" value="<?php echo $fila["nombre"]; ?>">
  <input type="submit" value="Guardar Cambios" name="enviar">
</form>
<?php 
}    
?>
 

Dado que ahí vuelvo a obtener el $ID.

Pero sigo teniendo el error de sintaxis:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '' at line 1

Me podrán ayudar?

Muchas gracias!

[bookmaster]

Dentro del if usa $_POST en vez de $_GET, y en el form mándalo por un input hidden. Te lo puso Drowna en su código.

[galaktico]

Al final lo pude resolver de esta manera:

Código PHP:

<?php
include("conexion.php");
 
if(isset($_GET['id_producto'])){
$id = $_GET['id_producto'];
$consulta = mysql_query("SELECT * FROM productos WHERE id_producto = $id", $conexion) or die(mysql_error());
$fila = mysql_fetch_array($consulta);
}
if(isset($_POST['enviar']) && $_POST['enviar'] == 'Enviar'){
if(!empty($_POST['nombre'])){
$id_producto = $_POST['id_producto'];
$nombre = $_POST['nombre'];
$consulta_update = mysql_query("UPDATE productos SET nombre = '$nombre' WHERE id_producto = '$id_producto'", $conexion)or die(mysql_error());
echo "Registro actualizado correctamente";
}else{
echo "debe llenar todos los campos";
}
}else{
?>
<form method="post" action="<?php $_SERVER['PHP_SELF']; ?>">
<input type="text" name="nombre" value="<?php echo $fila['nombre']; ?>" />
<input type="hidden" name="id_producto" value="<?php echo $fila['id_producto']; ?>" />
<input type="submit" name="enviar" value="Enviar" />
</form>
<?php } ?>
 

Saludos!

[gnzsoloyo]

Iniciado por galaktico

Yo parto de un buscador y luego de la página resultado paso el ID a modificar por GET, luego el formulario me levanta los datos cargados en la base y allí los modifico, pero después que? si lo hago con un archivo por separado pierdo el ID y si lo hago en el mismo archivo lo tengo vacío al ID.

Me parece que sigo sin entenderlo conceptualmente.

Vamos a ver si se entiende...

Supongamos que $_GET['id_producto'] llegue como NULL, es decir, o no llegue, o no contenga nada, o contenga un espacio en blanco o vacío.
Entonces esto:

Código PHP:

  $id = $_GET['id_producto'];  
   

Cuando hagas esto:

Código PHP:

  $consulta = "UPDATE productos SET nombre='$nombre' WHERE id_producto=$id";  
   

Generará la query para MySQL quedando así:

Código MySQL:

UPDATE productos SET nombre='$nombre' WHERE id_producto=
 

Y en este caso:

Código PHP:

  $consulta = "SELECT * FROM productos WHERE id_producto=$id";  
   

Quedará así:

Código MySQL:

SELECT * FROM productos WHERE id_producto=
 

¿Ves como en ambos casos le falta el valor de id_producto, y el segundo término de la comparación está vacío?
Bueno, eso es lo que genera un error de sintaxis.

Recuerda que MySQL no recibe variables de PHP, sino una cadena de texto de una consulta SQL. MySQL no entiende pHP. No sabe que le estás mandando algo, porque no le llega.

MySQL (y todo DBMS) es independiente de PHP y de cualquier lenguaje. Es un sistema separado.