Skip to main content
3 de 3
se añadieron 188 caracteres en el cuerpo
A. Cedano
  • 95.2k
  • 31
  • 145
  • 257

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.

A. Cedano
  • 95.2k
  • 31
  • 145
  • 257