Es preciso que comprendas que en las consultas preparadas se ponen marcadores, no los valores.
Por tanto esto es erróneo:
$stmt = Conexion::conectar()->prepare("UPDATE $tabla SET Hora_Cierre = :$Hora_Cierre, Total_Ventas = :$Total_Ventas, Estado = :$Estado WHERE Sucursal = :$Sucursal AND caja_sucursal = :$cajaAbierta");
Precisamente, el quid de una consulta preparada consiste en poner un marcador en el lugar donde va cada dato y luego, con un método apropiado (bindParam
en este caso) pasas el valor para cada dato asociado a su marcador. Es como decirle al controlador: donde puse este marcador, debes poner este dato, pero revisa que no haya riesgo de inyección.
Entendido eso, tu código debería quedar así:
$sql="UPDATE $tabla SET
Hora_Cierre = :Hora_Cierre,
Total_Ventas = :Total_Ventas,
Estado = :Estado
WHERE
Sucursal = :Sucursal AND
caja_sucursal = :caja_sucursal";
$stmt = Conexion::conectar()->prepare($sql);
Simplemente, hemos quitado toooodas las $
, teniendo así lo que se requiere, que son marcadores.
Ahora, debes decirle al SGBD a qué dato va asociado cada marcador que pusiste más arriba, algo MUY IMPORTANTE es que debes escribir cada marcador exactamente igual que como lo escribiste en la instrucción SQL, y debes vigilar que asocias en cada caso el valor correspondiente y no otro valor. Por tanto, si estás recibiendo estas variables en parámetro:
$Hora_Cierre
$Total_Ventas
$Estado
$cajaAbierta
$Sucursal
Debes hacer las vinculaciones usando esas variables:
$stmt -> bindParam(":Hora_Cierre", $Hora_Cierre, PDO::PARAM_STR);
$stmt -> bindParam(":Total_Ventas", $Total_Ventas, PDO::PARAM_STR);
$stmt -> bindParam(":Estado", $Estado, PDO::PARAM_STR);
$stmt -> bindParam(":Sucursal", $Sucursal, PDO::PARAM_STR);
$stmt -> bindParam(":caja_sucursal", $cajaAbierta, PDO::PARAM_STR);
No estabas vinculando :Sucursal
ni :caja_sucursal
. Yo he asumido que también son cadenas, si alguna no lo es, cambia PDO::PARAM_STR
por el tipo de dato correcto.
Así debería funcionar, salvo otros errores.
Advertencia sobre una mala práctica
En tu método, estas dos líneas al final no se ejecutarán nunca:
$stmt -> close();
$stmt = null;
¿Por quéeeee si las he puesto con muy buena intención? Pues porque tienes return
más arriba y nada lo que esté después de return
se ejecuta.
Además, son dos líneas redundantes, bastaría con una (de hecho, en algún escenario, no recuerdo ahora si con mysqli o con PDO), invocar a close()
no me destruía al objeto, null
en cambio siempre lo destruía.
Quizá una forma mejor de estructurar tu método sería esta:
static public function mdlCerrarCaja($tabla,$Hora_Cierre,$Total_Ventas,$Estado,$cajaAbierta,$Sucursal){
if($cajaAbierta!= null){
//Consulta UPDATE y demás...
if($stmt->execute()){
$mData=array(
"success"=>true,
"action"=>"update",
"data"=>"Filas actualizadas N..."
);
}else{
$mData=array(
"success"=>false,
"msg"=>"Error ... Conviene verificar el no de Error con errorInfo() y lanzar tu política de errores"
);
}
}else{
$stmt = Conexion::conectar()->prepare("SELECT * FROM $tabla");
$stmt -> execute();
$mData=array(
"success"=>true,
"action"=>"read",
"data"=>$stmt -> fetchAll()
);
}
$stmt = null;
$return $mData;
}
Si observas, se ha usado un objeto $mData
para controlar la lógica interna del método. Devolviendo una clave success
establecida a true
cuando haya algún dato o false
cuando haya algún error.
Dado que este método puede devolver dos tipos de respuestas exitosas, he puesto una clave action
por la cual podrás distinguir si ocurrió un update
o un read
y acoplar en la respuesta: mostrar el mensaje de cantidad de filas actualizadas por ejemplo, o leer los datos del fetchAll
que obtuviste en el SELECT
.
En cuanto a los casos de error, cuando success
sea false
el mensaje de lo ocurrido se encontrará en una clave msg
.
errorInfo()
.