Además, son dos líneas redundantes, bastaría con una (de hecho, en algún escenario, no recurdorecuerdo ahora si con mysqli o con PDO), invocar a close()
no me destruía al objeto, null
en cambio siempre lo destruía.
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()
);
}
$return$stmt $mData;
= null;
$stmt =$return null;$mData;
}
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
.