El problema es que no hay coherencia en tu consulta SELECT
, ya que en el execute
estás pasando un parámetro :Estado
que no has indicado en el prepare
. Imagino que quieres filtrar por el valor que se pasó en la variable $Estado
. Entonces tienes que agregar ese parámetro, que ahora mismo está escrito a mano con un valor 3
.
Aparte de eso, cuando escribas funciones conviene establecer un control estricto en ellas, de modo que puedas saber fácilmente qué ha ocurrido en tu función. Aquí he implementado una práctica que consiste en recoger la información dentro de un array $response
con dos claves: status
y msg
. Luego, en el resultado de la función puedes verificar status
y si quieres mostrar el mensaje de lo que ocurrió, hechas mano de msg
.
Otra cosa es que, si la finalidad del SELECT
es únicamente saber si el registro existe, es mejor hacer una consulta basa en COUNT
y luego obtener la cantidad de filas, usando fetchColumn()
. Es más ligero y más seguro que hacer un fetchAll
con unos datos que al fin y al cabo no vas a usar.
Aplicando todo eso el código quedaría así.
<?php
public function addCita($Next,$Trabajadora,$Cliente,$Concat,$Suma,$Observaciones,$Estado)
{
$sql= "SELECT COUNT(*) from cita WHERE Id_Trabajadora= :Trabajadora and Fecha_Hora= :Concat and Id_Estado !=:Estado";
$query = $this->db->prepare($sql);
$parameters = array(':Trabajadora' => $Trabajadora,':Concat' => $Concat, ':Estado' => $Estado);
if ( !$query->execute($parameters) )
{
$response=array("status"=>FALSE, "msg"=>"Error en la consulta SELECT");
}
else
{
if ( $query->fetchColumn()>0 )
{
$response=array("status"=>FALSE, "msg"=>"$Trabajadora existe");
}
else
{
$sql = "INSERT INTO `cita`(`Id_Cita`, `Id_Trabajadora`, `Id_Cliente`, `Fecha_Hora`, `Precio_Total`, `Observaciones`, `Id_Estado`) VALUES (:Next, :Trabajadora, :Cliente, :Concat, :Suma, :Observaciones, :Estado)";
$query = $this->db->prepare($sql);
$parameters = array(':Next' => $Next, ':Trabajadora' => $Trabajadora, ':Cliente' => $Cliente, ':Concat' => $Concat, ':Suma' => $Suma, ':Observaciones' => $Observaciones, ':Estado' => $Estado);
if ( !$query->execute($parameters) )
{
$response=array("status"=>FALSE, "msg"=>"Error en la consulta INSERT");
}
else
{
$response=array("status"=>TRUE, "msg"=>"$Trabajadora insertada");
}
}
}
return $response;
}
?>
##Ejemplo de uso
$mData=addCita (....aquí los parámetros...);
if ( $mData["status") )
{
//Se insertó el registro
}
else
{
echo "Hubo un error: $mData[msg]";
}