Hay varias cosas que puedes hacer para mejorar el código si tienes una versión reciente de PHP. Mira:
public function existeProyectoActivo($usuario) {
self::accesoDB();
try {
$consulta = "SELECT nombre FROM Proyecto WHERE usuario = :usuario and"
. " activo = true LIMIT 1";
$resultado = self::$conexion->prepare($consulta);
$resultado->bindParam(":usuario", $usuario);
$resultado->execute();
if ($resultado->rowCount() > 0) {
$row = $resultado->fetch();
$nombre = (string) $row['nombre'];
}
return $nombre ?? '';
} catch (PDOExceptio $e) {
echo "Código de error: " . $e->getCode() . "<br />";
echo "Mensaje: " . $e->getMessage() . "<br />";
}
}
En primer lugar, fíjate que es importante limitar el número de resultados devueltos por MySQL poniendo LIMIT 1
en la consulta SQL. Así, te aseguras que tú código siempre vaya a funcionar aunque tu base de datos se corrompa.
En segundo lugar, ¿ por qué devolver null
?. Hay que ser coherentes en la medida de lo posible y si tu función devuelve una cadena cuando cuando el valor es correcto, también debería de devolver una cadena en caso incorrecto. Fíjate hemos eliminado la asignación a null
y la devolución ahora es:
return $nombre ?? '';
Esto significa que si nombre no existiera que se use cadena vacía( '' ).
Luego en el javascript:
function existeProyectoActivo() {
$.post("funciones.php", {orden: 'existeProyectoActivo'},
function (nombre) {
console.log(nombre);
if ('' === nombre) {
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.");
}
});
}
Fíjate, ahora se comprueba contra cadena vacía y no contra null. Ahora debería de funcionar corréctamente.
¡ Suerte !