Fix: Implicitly marking parameter as nullable is deprecated.

PHP 8.4 deprecation.

index.php

@@ -7,18 +7,14 @@ spl_autoload_register(function ($className) {
     $name = basename($fp);
     $dir  = dirname($fp);
     $file = ROOT_CORE.'/'.$dir.'/'.$name.'.php';
-    if (file_exists($file)) {
+    if (file_exists($file))
         require_once $file;
-        return;
-    }
 });
 
 // Incluir routers
 $routers = glob(ROOT_CORE.'/Routers/*.php');
 
-foreach($routers as $file){
+foreach($routers as $file)
     require_once($file);
-}
 
 \Libs\Router::apply();
-?>

src/Libs/Model.php

@@ -85,7 +85,10 @@ class Model {
      */
     public function rollBack(): bool
     {
-        return static::db()->rollBack();
+        if ( static::db()->inTransaction())
+            return static::db()->rollBack();
+        else
+            return true;
     }
 
     /**
@@ -96,7 +99,10 @@ class Model {
      */
     public function commit(): bool
     {
-        return static::db()->commit();
+        if (static::db()->inTransaction())
+            return static::db()->commit();
+        else
+            return true;
     }
 
     /**
@@ -129,7 +135,6 @@ class Model {
 
             $vars = json_encode(static::$queryVars);
 
-            echo "<pre>";
             throw new Exception(
                 "\nError at query to database.\n" .
                 "Query: $query\n" .
@@ -236,11 +241,18 @@ class Model {
      */
     protected static function getInstance(array $elem = []): static
     {
-        $class = get_called_class();
+        $class         = get_called_class();
-        $instance = new $class;
+        $instance      = new $class;
+        $reflection    = new ReflectionClass($instance);
+        $properties    = $reflection->getProperties();
+        $propertyNames = array_column($properties, 'name');
 
         foreach ($elem as $key => $value) {
-            $instance->$key = $value;
+            $index = array_search($key, $propertyNames);
+            if (is_numeric($index) && enum_exists($properties[$index]->getType()->getName()))
+                $instance->$key = $properties[$index]->getType()->getName()::tryfrom($value);
+            else
+                $instance->$key = $value;
         }
 
         return $instance;
@@ -272,10 +284,14 @@ class Model {
             $result[$value] = isset($this->$value)
                             ? $this->$value: null;
 
-        foreach ($result as $i => $property)
+        foreach ($result as $i => $property) {
             if (gettype($property) == 'boolean')
                 $result[$i] = $property ? '1' : '0';
 
+            if ($property instanceof \UnitEnum)
+                $result[$i] = $property->value;
+        }
+
         return $result;
     }
 
@@ -287,13 +303,8 @@ class Model {
      */
     public static function className(): string
     {
-        return strtolower(
+        return substr(
-            preg_replace(
+            strrchr(get_called_class(), '\\'), 1
-                '/(?<!^)[A-Z]/', '_$0',
-                substr(
-                    strrchr(get_called_class(), '\\'), 1
-                )
-            )
         );
     }
 
@@ -308,7 +319,13 @@ class Model {
     {
         if (isset(static::$table))
             return static::$table;
-        return static::className().static::$tableSufix;
+
+        return strtolower(
+            preg_replace(
+                '/(?<!^)[A-Z]/', '_$0',
+                static::className()
+            )
+        ).static::$tableSufix;
     }
 
     /**
@@ -432,7 +449,7 @@ class Model {
      * @param string $operatorOrValue
      *   El operador o el valor a comparar como igual en caso de que $value no se defina.
      *
-     * @param string $value
+     * @param string|null $value
      *   (opcional) El valor a comparar en la columna.
      *
      * @param bool $no_filter
@@ -442,10 +459,10 @@ class Model {
      * @return static
      */
     public static function where(
-        string $column,
+        string  $column,
-        string $operatorOrValue,
+        string  $operatorOrValue,
-        string $value   = null,
+        ?string $value     = null,
-        bool $no_filter = false
+        bool    $no_filter = false
     ): static
     {
         return static::and(
@@ -465,7 +482,7 @@ class Model {
      * @param string $operatorOrValue
      *   El operador o el valor a comparar como igual en caso de que $value no se defina.
      *
-     * @param string $value
+     * @param string|null $value
      *   (opcional) El valor el valor a comparar en la columna.
      *
      * @param bool $no_filter
@@ -475,10 +492,10 @@ class Model {
      * @return static
      */
     public static function and(
-        string $column,
+        string  $column,
-        string $operatorOrValue,
+        string  $operatorOrValue,
-        string $value   = null,
+        ?string $value     = null,
-        bool $no_filter = false
+        bool    $no_filter = false
     ): static
     {
         if (is_null($value)) {
@@ -506,7 +523,7 @@ class Model {
      * @param string $operatorOrValue
      *   El operador o el valor a comparar como igual en caso de que $value no se defina.
      *
-     * @param string $value
+     * @param string|null $value
      *   (opcional) El valor el valor a comparar en la columna.
      *
      * @param bool $no_filter
@@ -516,10 +533,10 @@ class Model {
      * @return static
      */
     public static function or(
-        string $column,
+        string  $column,
-        string $operatorOrValue,
+        string  $operatorOrValue,
-        string $value   = null,
+        ?string $value   = null,
-        bool $no_filter = false
+        bool    $no_filter = false
     ): static
     {
         if (is_null($value)) {
@@ -564,9 +581,14 @@ class Model {
         }
 
         if ($in)
-            static::$querySelect['where'] = "$column IN (".join(', ', $arrIn).")";
+            $where_in = "$column IN (".join(', ', $arrIn).")";
         else
-            static::$querySelect['where'] = "$column NOT IN (".join(', ', $arrIn).")";
+            $where_in = "$column NOT IN (".join(', ', $arrIn).")";
+
+        if (static::$querySelect['where'] == '')
+            static::$querySelect['where'] = $where_in;
+        else
+            static::$querySelect['where'] .= " AND $where_in";
 
         return new static();
     }
@@ -583,16 +605,16 @@ class Model {
      * @param string $operatorOrColumnB
      *   Operador o columna a comparar como igual para hacer el join en caso de que $columnB no se defina.
      *
-     * @param string $columnB
+     * @param string|null $columnB
      *   (opcional) Columna a comparar para hacer el join.
      *
      * @return static
      */
     public static function leftJoin(
-        string $table,
+        string  $table,
-        string $columnA,
+        string  $columnA,
-        string $operatorOrColumnB,
+        string  $operatorOrColumnB,
-        string $columnB = null
+        ?string $columnB = null
     ): static
     {
         if (is_null($columnB)) {
@@ -617,16 +639,16 @@ class Model {
      * @param string $operatorOrColumnB
      *   Operador o columna a comparar como igual para hacer el join en caso de que $columnB no se defina.
      *
-     * @param string $columnB
+     * @param string|null $columnB
      *   (opcional) Columna a comparar para hacer el join.
      *
      * @return static
      */
     public static function rightJoin(
-        string $table,
+        string  $table,
-        string $columnA,
+        string  $columnA,
-        string $operatorOrColumnB,
+        string  $operatorOrColumnB,
-        string $columnB = null
+        ?string $columnB = null
     ): static
     {
         if (is_null($columnB)) {
@@ -651,16 +673,16 @@ class Model {
      * @param string $operatorOrColumnB
      *   Operador o columna a comparar como igual para hacer el join en caso de que $columnB no se defina.
      *
-     * @param string $columnB
+     * @param string|null $columnB
      *   (opcional) Columna a comparar para hacer el join.
      *
      * @return static
      */
     public static function innerJoin(
-        string $table,
+        string  $table,
-        string $columnA,
+        string  $columnA,
-        string $operatorOrColumnB,
+        string  $operatorOrColumnB,
-        string $columnB = null
+        ?string $columnB = null
     ): static
     {
         if (is_null($columnB)) {
@@ -801,20 +823,18 @@ class Model {
      * @param string $search
      *   Contenido a buscar.
      *
-     * @param array $in
+     * @param array|null $in
      *   (opcional) Columnas en las que se va a buscar (null para buscar en todas).
      *
      * @return static
      */
-    public static function search(string $search, array $in = null): static
+    public static function search(string $search, ?array $in = null): static
     {
         if ($in == null) {
             $className = get_called_class();
             $in = array_keys((new $className())->getVars());
         }
 
-        $db = static::db();
-
         $search = static::bindValue($search);
         $where  = [];
 
@@ -840,7 +860,7 @@ class Model {
      * @param bool $resetQuery
      *   (opcional) Indica si el query debe reiniciarse o no (por defecto es true).
      *
-     * @return array
+     * @return array<static>
      *   Arreglo con instancias del la clase actual resultantes del query.
      */
     public static function get(bool $resetQuery = true): array
@@ -876,7 +896,7 @@ class Model {
     /**
      * Obtener todos los elementos del la tabla de la instancia actual.
      *
-     * @return array
+     * @return array<static>
      *   Contiene un arreglo de instancias de la clase actual.
      */
     public static function all(): array

src/Libs/Request.php

@@ -15,10 +15,14 @@ namespace Libs;
 class Request extends Neuron  {
     public Neuron $get;
     public Neuron $post;
+    public Neuron $put;
+    public Neuron $patch;
+    public Neuron $delete;
     public Neuron $json;
     public Neuron $params;
     public string $path;
     public string $error;
+    public string $body;
     public array  $next;
 
     /**
@@ -31,34 +35,57 @@ class Request extends Neuron  {
         $this->path   = Router::currentPath();
         $this->get    = new Neuron($_GET);
         $this->post   = new Neuron($_POST);
+        $this->put    = new Neuron();
+        $this->patch  = new Neuron();
+        $this->delete = new Neuron();
+        $this->body   = file_get_contents("php://input");
 
         $contentType = isset($_SERVER["CONTENT_TYPE"]) ? trim($_SERVER["CONTENT_TYPE"]) : '';
         if ($contentType === "application/json")
             $this->json = new Neuron(
-                (object) json_decode(trim(file_get_contents("php://input")), false)
+                (object) json_decode(trim($this->body), false)
             );
-        else
+        else {
-            $this->json = new Neuron();
+            $this->json   = new Neuron();
+            if (in_array($_SERVER['REQUEST_METHOD'], ['PUT', 'PATCH', 'DELETE']) &&
+                preg_match('/^[^;?\/:@&=+$,]{1,255}[=]/', $this->body, $matches)) { // Con la expresión regular verificamos que sea un http query string válido y evitamos errores de memoria en caso de que el body tenga algo más grande que eso.
+                parse_str($this->body, $input_vars);
+                $this->{strtolower($_SERVER['REQUEST_METHOD'])} = new Neuron($input_vars);
+            }
+        }
 
         $this->params = new Neuron();
     }
 
     /**
-     * Inicia la validación que se haya configurado.
+     * Corre las validaciones e intenta continuar con la pila de callbacks.
      *
      * @return mixed
      */
-    public function validate(): mixed
+    public function handle(): mixed
     {
-        if ($_SERVER['REQUEST_METHOD'] == 'GET')
+        if ($this->validate())
-            $actual = $this->get;
+            return Middleware::next($this);
-        else
+
-            $actual = $this->post;
+        return null;
+    }
+
+    /**
+     * Inicia la validación que se haya configurado.
+     *
+     * @return bool
+     */
+    public function validate(): bool
+    {
+        $actual = match($_SERVER['REQUEST_METHOD']) {
+            'POST', 'PUT', 'PATCH', 'DELETE' => $this->{strtolower($_SERVER['REQUEST_METHOD'])},
+            default => $this->get
+        };
 
         if (Validator::validateList(static::paramRules(), $this->params) &&
             Validator::validateList(static::getRules(),   $this->get   ) &&
             Validator::validateList(static::rules(),      $actual))
-            return Middleware::next($this);
+            return true;
 
         if (isset(static::messages()[Validator::$lastFailed]))
             $error =  static::messages()[Validator::$lastFailed];
@@ -67,7 +94,8 @@ class Request extends Neuron  {
             $error = 'Error: validation failed of '.preg_replace('/\./', ' as ', Validator::$lastFailed, 1);
         }
 
-        return static::onInvalid($error);
+        static::onInvalid($error);
+        return false;
     }
 
     /**
@@ -75,7 +103,7 @@ class Request extends Neuron  {
      *
      * @return array
      */
-    public static function rules(): array {
+    public function rules(): array {
         return [];
     }
 
@@ -84,7 +112,7 @@ class Request extends Neuron  {
      *
      * @return array
      */
-    public static function paramRules(): array {
+    public function paramRules(): array {
         return [];
     }
 
@@ -93,7 +121,7 @@ class Request extends Neuron  {
      *
      * @return array
      */
-    public static function getRules(): array {
+    public function getRules(): array {
         return [];
     }
 
@@ -102,7 +130,7 @@ class Request extends Neuron  {
      *
      * @return array
      */
-    public static function messages(): array {
+    public function messages(): array {
         return [];
     }
 
@@ -111,12 +139,11 @@ class Request extends Neuron  {
      *
      * @param string $error
      *
-     * @return mixed
+     * @return void
      */
-    protected function onInvalid(string $error): mixed
+    public function onInvalid(string $error): void
     {
         http_response_code(422);
         print($error);
-        return null;
     }
 }

src/Libs/Router.php

@@ -114,7 +114,7 @@ class Router {
      * @return static
      *   Devuelve la instancia actual.
      */
-    public static function middleware(callable $callback, int $priority = null): static
+    public static function middleware(callable $callback, ?int $priority = null): static
     {
         if (!isset(static::$last))
             return new static();
@@ -202,13 +202,13 @@ class Router {
      *
      * @param string $path
      *   Ruta con pseudovariables.
-     * @param callable $callback
+     * @param callable|null $callback
      *   Callback que será llamado cuando la ruta configurada en $path coincida.
      *
      * @return static
      *   Devuelve la instancia actual.
      */
-    public static function get(string $path, callable $callback = null): static
+    public static function get(string $path, ?callable $callback = null): static
     {
         return static::configure('get', $path, $callback);
     }
@@ -218,13 +218,13 @@ class Router {
      *
      * @param string $path
      *   Ruta con pseudovariables.
-     * @param callable $callback
+     * @param callable|null $callback
      *   Callback que será llamado cuando la ruta configurada en $path coincida.
      *
      * @return static
      *   Devuelve la instancia actual.
      */
-    public static function post(string $path, callable $callback = null): static
+    public static function post(string $path, ?callable $callback = null): static
     {
         return static::configure('post', $path, $callback);
     }
@@ -234,14 +234,14 @@ class Router {
      *
      * @param string $path
      *   Ruta con pseudovariables.
-     * @param callable $callback
+     * @param callable|null $callback
      *   Callback que será llamado cuando la ruta configurada en $path coincida.
      *
      * @return static
      *   Devuelve la instancia actual
      */
 
-    public static function put(string $path, callable $callback = null): static
+    public static function put(string $path, ?callable $callback = null): static
     {
         return static::configure('put', $path, $callback);
     }
@@ -251,13 +251,13 @@ class Router {
      *
      * @param string $path
      *   Ruta con pseudovariables.
-     * @param callable $callback
+     * @param callable|null $callback
      *   Callback que será llamado cuando la ruta configurada en $path coincida.
      *
      * @return static
      *   Devuelve la instancia actual
      */
-    public static function patch(string $path, callable $callback = null): static
+    public static function patch(string $path, ?callable $callback = null): static
     {
         return static::configure('patch', $path, $callback);
     }
@@ -267,13 +267,13 @@ class Router {
      *
      * @param string $path
      *   Ruta con pseudovariables
-     * @param callable $callback
+     * @param callable|null $callback
      *   Callback que será llamado cuando la ruta configurada en $path coincida.
      *
      * @return static
      *   Devuelve la instancia actual
      */
-    public static function delete(string $path, callable $callback = null): static
+    public static function delete(string $path, ?callable $callback = null): static
     {
         return static::configure('delete', $path, $callback);
     }
@@ -292,11 +292,11 @@ class Router {
     /**
      * Aplica la configuración de rutas.
      *
-     * @param string $path (opcional) Ruta a usar. Si no se define, detecta la ruta actual.
+     * @param string|null $path (opcional) Ruta a usar. Si no se define, detecta la ruta actual.
      *
      * @return void
      */
-    public static function apply(string $path = null): void
+    public static function apply(?string $path = null): void
     {
         $path    = $path ?? static::currentPath();
         $routers = match($_SERVER['REQUEST_METHOD']) { // Según el método selecciona un arreglo de routers
@@ -356,7 +356,7 @@ class Router {
 
                 // Llama a la validación y luego procesa la cola de callbacks
                 $request->next = $router['callback'];
-                $data          = $request->validate($request);
+                $data          = $request->handle();
 
                 // Por defecto imprime como JSON si se retorna algo
                 if (isset($data)) {

src/Libs/Validator.php

@@ -11,7 +11,8 @@
  * | Regla    | Descripción                                            |
  * |----------+--------------------------------------------------------|
  * | not      | Niega la siguiente regla. Ej: not:float                |
- * | required | Es requerido                                           |
+ * | exists   | Es requerido; debe estar definido y puede estar vacío  |
+ * | required | Es requerido; debe estar definido y no vacío           |
  * | number   | Es numérico                                            |
  * | int      | Es entero                                              |
  * | float    | Es un float                                            |
@@ -90,7 +91,19 @@ class Validator {
     }
 
     /**
-     * required
+     * Comprueba que que esté definido/exista.
+     *
+     * @param mixed $subject
+     *
+     * @return bool
+     */
+    public static function exists(mixed $subject): bool
+    {
+        return isset($subject);
+    }
+
+    /**
+     * Comprueba que que esté definido y no esté vacío.
      *
      * @param mixed $subject
      *
@@ -98,7 +111,7 @@ class Validator {
      */
     public static function required(mixed $subject): bool
     {
-        return isset($subject);
+        return isset($subject) && !empty($subject);
     }
 
     /**

src/Libs/View.php

@@ -16,16 +16,16 @@ class View extends Neuron {
     /**
      * Incluye el archivo.
      *
-     * @param string $viewName  Ruta relativa y el nommbre sin extensión del archivo.
+     * @param string      $viewName  Ruta relativa y el nommbre sin extensión del archivo.
-     * @param string $viewPath  (opcional) Ruta donde se encuentra la vista.
+     * @param string|null $viewPath  (opcional) Ruta donde se encuentra la vista.
-     * @param string $extension (opcional) Extensión del archivo.
+     * @param string      $extension (opcional) Extensión del archivo.
      *
      * @return void
      */
-    private function include(
+    protected function include(
-        string $viewName,
+        string  $viewName,
-        string $viewPath  = null,
+        ?string $viewPath  = null,
-        string $extension = 'php'
+        string  $extension = 'php'
     ): void
     {
         $view = $this;
@@ -44,7 +44,7 @@ class View extends Neuron {
      *
      * @param string        $viewName  Ruta relativa y el nommbre sin extensión del archivo.
      * @param array|Neuron  $params    (opcional) Arreglo que podrá ser usado en la vista mediante $view ($param['index'] se usaría así: $view->index)
-     * @param string        $viewPath  (opcional) Ruta donde se encuentra la vista. En caso de que la vista no se encuentre en esa ruta, se usará la ruta por defecto "src/Views/".
+     * @param string|null   $viewPath  (opcional) Ruta donde se encuentra la vista. En caso de que la vista no se encuentre en esa ruta, se usará la ruta por defecto "src/Views/".
      * @param string        $extension (opcional) Extensión del archivo.
      *
      * @return void
@@ -52,27 +52,27 @@ class View extends Neuron {
     public static function render(
         string $viewName,
         array|Neuron $params = [],
-        string $viewPath     = null,
+        ?string $viewPath    = null,
         string $extension    = 'php'
     ): void
     {
         $instance = new View($params);
-        $instance->html($viewName, $viewPath);
+        $instance->html($viewName, $viewPath, $extension);
     }
 
     /**
      * Renderiza las vistas HTML
      *
-     * @param string $viewName  Ruta relativa y el nommbre sin extensión del archivo ubicado en src/Views
+     * @param string      $viewName  Ruta relativa y el nommbre sin extensión del archivo ubicado en src/Views
-     * @param string $viewPath  (opcional) Ruta donde se encuentra la vista. En caso de que la vista no se encuentre en esa ruta, se usará la ruta por defecto "src/Views/".
+     * @param string|null $viewPath  (opcional) Ruta donde se encuentra la vista. En caso de que la vista no se encuentre en esa ruta, se usará la ruta por defecto "src/Views/".
-     * @param string $extension (opcional) Extensión del archivo.
+     * @param string      $extension (opcional) Extensión del archivo.
      *
      * @return void
      */
     public function html(
-        string $viewName,
+        string  $viewName,
-        string $viewPath  = null,
+        ?string $viewPath  = null,
-        string $extension = 'php'
+        string  $extension = 'php'
     ): void
     {
         $this->include(
@@ -85,16 +85,16 @@ class View extends Neuron {
     /**
      * Renderiza código CSS.
      *
-     * @param string $viewName  Ruta relativa y el nommbre sin extensión del archivo ubicado en src/Views
+     * @param string      $viewName  Ruta relativa y el nommbre sin extensión del archivo ubicado en src/Views
-     * @param string $viewPath  (opcional) Ruta donde se encuentra la vista. En caso de que la vista no se encuentre en esa ruta, se usará la ruta por defecto "src/Views/".
+     * @param string|null $viewPath  (opcional) Ruta donde se encuentra la vista. En caso de que la vista no se encuentre en esa ruta, se usará la ruta por defecto "src/Views/".
-     * @param string $extension (opcional) Extensión del archivo.
+     * @param string      $extension (opcional) Extensión del archivo.
      *
      * @return void
      */
     public function css(
-        string $viewName,
+        string  $viewName,
-        string $viewPath  = null,
+        ?string $viewPath  = null,
-        string $extension = 'css'
+        string  $extension = 'css'
     ): void
     {
         header("Content-type: text/css");
@@ -104,16 +104,16 @@ class View extends Neuron {
     /**
      * Renderiza código Javascript.
      *
-     * @param string $viewName  Ruta relativa y el nommbre sin extensión del archivo ubicado en src/Views
+     * @param string      $viewName  Ruta relativa y el nommbre sin extensión del archivo ubicado en src/Views
-     * @param string $viewPath  (opcional) Ruta donde se encuentra la vista. En caso de que la vista no se encuentre en esa ruta, se usará la ruta por defecto "src/Views/".
+     * @param string|null $viewPath  (opcional) Ruta donde se encuentra la vista. En caso de que la vista no se encuentre en esa ruta, se usará la ruta por defecto "src/Views/".
-     * @param string $extension (opcional) Extensión del archivo.
+     * @param string      $extension (opcional) Extensión del archivo.
      *
      * @return void
      */
     public function js(
-        string $viewName,
+        string  $viewName,
-        string $viewPath  = null,
+        ?string $viewPath  = null,
-        string $extension = 'js'
+        string  $extension = 'js'
     ): void
     {
         header("Content-type: application/javascript");
@@ -153,7 +153,7 @@ class View extends Neuron {
      *
      * @return string
      */
-    public function route(string $path = '/'): string
+    public static function route(string $path = '/'): string
     {
         if (defined('SITE_URL') && !empty(SITE_URL))
             return rtrim(SITE_URL, '/').'/'.ltrim($path, '/');