Tu UPDATE es peligroso y aunque crees que no pase nada, puede que pase algo muy grave: que se estén actualizando los registros que no son.
Las variables debes pasarlas en su orden, más aún si estás usando bindValue
con marcadores de posición (?
).
He resaltado posición a propósito, porque aquí la posición es vital.
El Manual dice hablando del orden que se debe observar cuando usamos bindValue
dice:
Para sentencias preparadas que usen parámetros de sustición de signos
de interrogación, esto [el número usado con bindValue] será la posición índice-1 del parámetro.
Si tu consulta está escrita así:
$producto = $conn->prepare
("UPDATE productos SET nombre = ?, precio = ?,
marca = ?, presentacion = ?, stock = ? WHERE idproducto = ?");
Y usas bindValue
debes pasar los valores en este orden:
- nombre
- precio
- marca
- presentacion
- stock
- id_producto
Es decir:
$producto->bindValue(1, $nombre);
$producto->bindValue(2, $precio);
$producto->bindValue(3, $marca);
$producto->bindValue(4, $presentacion);
$producto->bindValue(5, $stock);
$producto->bindValue(6, $id);
Pero tú estás pasando primero, por lo que veo, el valor que debería corresponder a id_producto
...
$producto->bindValue(1, $id); //Este es el último ? de tu consulta ¿sí o no? No debería ser el 1º aquí
$producto->bindValue(2, $nombre);
$producto->bindValue(3, $precio);
$producto->bindValue(4, $marca);
$producto->bindValue(5, $presentacion);
$producto->bindValue(6, $stock);
Lo que ocurre aquí es que en tu UPDATE
nombre será igual a $id
, precio será igual a nombre (o a cero, quien sabe), marca será igual a precio... y así y el registro que se actualizará será el que tenga un id_producto
igual a $stock
Tu consulta si está funcionando, está actualizando los registros que no son. ¿Ves el peligro?
POST-DATA
Por si no lo sabías, con PDO puedes pasar totalmente de bindValue
, creando un array con tus valores y pasando ese array en el execute
.
Ejemplo:
global $conn; //¿globales?... oh no
$producto = $conn->prepare
("UPDATE productos SET nombre = ?, precio = ?,
marca = ?, presentacion = ?, stock = ? WHERE idproducto = ?");
/*array respetando el orden de cada valor*/
$arrParams=array($nombre,$precio,$marca,$presentacion,$stock,$id);
/*Pasamos el array en el execute*/
if($producto->execute($arrParams))
{
$respuesta = "El articulo se actualizo con éxito. Filas afectadas: ".$producto->rowCount();
header("location: ". BACK_END_URL ."/?rta=".$respuesta."&color=verde");
}
else
{
$respuesta = "El articulo no se actualizo con exito";
header("location: ". BACK_END_URL ."/?rta=".$respuesta."&color=rojo");
}
$producto->bindValue(1, $id);
El valor que usará comoWHERE
en tu código será el valor de$stock
que es el último. Si el código funciona podrías estar actualizando los registros con otro id... y fastidiar tus datos.