Skip to main content
Se elimina ruido.
Origen Enlace

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 !

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 !

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.

Origen Enlace

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 !