Aquí lo más seguro sería recoger en la respuesta dos posibilidades no cubiertas. Para hacerlo, puedes poner un else
para recoger la posibilidad de cero filas encontradas, generando ahí un array con una clave error
y un mensaje que puedes usar para informar al cliente de lo que ocurrió en el servidor. La posibilidad de error en la consulta se podría retornar también igualmente al final del bloque try ... catch
.
En tu código no puedes emitir echo
en ningún momento, porque si sale cualquier cosa por pantalla el json podría ser incorrecto en esos casos, y tu retorno a la función que llama no sería coherente.
También, te falta una n
al final aquí: PDOExceptio
. Y, conviene que especifiques el estilo fetch
de forma explícita, de lo contrario podría estar trayéndote los datos en forma de array asociativo y numérico.
Otra cosa (que se ha verificado posteriormente en comentarios) es que la respuesta en el cliente no está siendo reconocida como json, sino como texto. Convendría dar más coherencia al código, indicando en el cliente que se espera un tipo de datos json, y poniendo en el servidor un header
adecuado que indique que lo que se emite es un json (si te interesa hacerlo así puedo poner un ejemplo). De ese modo, no haría falta parsear los datos, lo cual implica posibles riesgos de producir un json inválido, por ejemplo cuando vienen datos con caracteres especiales debido a que la codificación en el servidor no es la más adecuada.
Puedes intentar así:
public function existeProyectoActivo($usuario) {
self::accesoDB();
try {
$consulta = "SELECT nombre FROM Proyecto WHERE usuario = :usuario and"
. " activo = true";
$resultado = self::$conexion->prepare($consulta);
$resultado->bindParam(":usuario", $usuario);
$resultado->execute();
if ($resultado->rowCount() > 0) {
$row = $resultado->fetch(PDO::FETCH_ASSOC);
$nombre = (string)$row['nombre'];
} else {
$nombre=["error"=>"No es encontraron filas"];
}
return $nombre;
} catch (PDOException $e) {
return ["error"=>"Hubo un error en la consulta"];
}
$nombre=["error"=>"La consulta falló"];
return $nombre;
}
Y en la verificación haces esta evaluación if (nombre.error) {
, quedando así el código:
function existeProyectoActivo() {
$.post("funciones.php", {orden: 'existeProyectoActivo'},
function (nombre) {
/*Esto sólo es necesario cuando no se ha indicado de otro modo el datatype*/
json=JSON.parse(nombre);
console.log(json);
/*Aquí comprobamos si el dato parseado tiene o no una clave error*/
if (json.error) {
alert("Existe otro proyecto activo. Si crea este proyecto con el \n\
campo \"Activo\" seleccionado, se desactivará el otro \n\
proyecto y se incluirá en él la fecha de ayer como fecha final.");
}
});
}
Así estandarizas, devolviendo siempre un json válido que contenga información útil para el cliente.
Ejemplo poniendo el dataType
En el cliente
Para que esto funcione necesitas tener una librería de jQuery actualizada. Considera agregar a tu proyecto la librería más reciente.
function existeProyectoActivo() {
$.post( "funciones.php", {orden: 'existeProyectoActivo'},
function( nombre ) {
if (json.error) {
alert("Existe otro proyecto activo. Si crea este proyecto con el \n\
campo \"Activo\" seleccionado, se desactivará el otro \n\
proyecto y se incluirá en él la fecha de ayer como fecha final.");
}
}, "json");
}
Haciendo eso ya no necesitas aplicar JSON.parse
como anteriormente.
En el servidor
Para completar el contexto, conviene que pongas un header
también en PHP antes de devolver los datos. OJO con esto, cuando usas header
en PHP debes cuidar que no salga por pantalla nada antes de eso.
case "existeProyectoActivo":
$nombreProyectoActivo = DB::existeProyectoActivo($_SESSION['usuario']->getLogin());
header("Content-type: application/json; charset=utf-8");
echo json_encode($nombreProyectoActivo);
break;
Dado que veo que estás en un contexto de switch ... case
quizá convendría organizar mejor el código para no estar repitiendo N
veces el header o los echo
. Eso se haría guardando el resultado en una variable e imprimiéndola al final del bloque switch
, eso no será difícil de adaptar.
$.post
espera un objeto JSON del tipo{ nombre: "Producto" }
de lo contrario no recibirá nada, ni mostrara nadaecho json_encode($nombreProyectoActivo);
if (nombre.error) {
para que determine si el json tiene una claveerror
o no. Si no funciona quizá tengas que aplicarJSON.parse
antes de hacer elif
como indiqué en uno de mis comentarios en mi respuesta. Si no funciona, por favor muestra lo que imprime la consola, a ver si no hay otro problema de codificación por ejemplo o si está devolviendo un objeto mal formado.