Fix: Implicitly marking parameter as nullable is deprecated.

PHP 8.4 deprecation.

index.php

@@ -18,4 +18,3 @@ foreach($routers as $file)
     require_once($file);
 
 \Libs\Router::apply();
-?>

src/Libs/Model.php

@@ -135,7 +135,6 @@ class Model {
 
             $vars = json_encode(static::$queryVars);
 
-            echo "<pre>";
             throw new Exception(
                 "\nError at query to database.\n" .
                 "Query: $query\n" .
@@ -450,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
@@ -462,7 +461,7 @@ class Model {
     public static function where(
         string  $column,
         string  $operatorOrValue,
-        string $value   = null,
+        ?string $value     = null,
         bool    $no_filter = false
     ): static
     {
@@ -483,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
@@ -495,7 +494,7 @@ class Model {
     public static function and(
         string  $column,
         string  $operatorOrValue,
-        string $value   = null,
+        ?string $value     = null,
         bool    $no_filter = false
     ): static
     {
@@ -524,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
@@ -536,7 +535,7 @@ class Model {
     public static function or(
         string  $column,
         string  $operatorOrValue,
-        string $value   = null,
+        ?string $value   = null,
         bool    $no_filter = false
     ): static
     {
@@ -606,7 +605,7 @@ 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
@@ -615,7 +614,7 @@ class Model {
         string  $table,
         string  $columnA,
         string  $operatorOrColumnB,
-        string $columnB = null
+        ?string $columnB = null
     ): static
     {
         if (is_null($columnB)) {
@@ -640,7 +639,7 @@ 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
@@ -649,7 +648,7 @@ class Model {
         string  $table,
         string  $columnA,
         string  $operatorOrColumnB,
-        string $columnB = null
+        ?string $columnB = null
     ): static
     {
         if (is_null($columnB)) {
@@ -674,7 +673,7 @@ 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
@@ -683,7 +682,7 @@ class Model {
         string  $table,
         string  $columnA,
         string  $operatorOrColumnB,
-        string $columnB = null
+        ?string $columnB = null
     ): static
     {
         if (is_null($columnB)) {
@@ -824,12 +823,12 @@ 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();

src/Libs/Request.php

@@ -22,6 +22,7 @@ class Request extends Neuron  {
     public Neuron $params;
     public string $path;
     public string $error;
+    public string $body;
     public array  $next;
 
     /**
@@ -37,16 +38,18 @@ class Request extends Neuron  {
         $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 {
             $this->json   = new Neuron();
-            if (in_array($_SERVER['REQUEST_METHOD'], ['PUT', 'PATCH', 'DELETE'])) {
-                parse_str(file_get_contents("php://input"), $input_vars);
+            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);
             }
         }
@@ -91,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;
     }
 
     /**
@@ -135,12 +139,11 @@ class Request extends Neuron  {
      *
      * @param string $error
      *
-     * @return false
+     * @return void
      */
-    public function onInvalid(string $error): false
+    public function onInvalid(string $error): void
     {
         http_response_code(422);
         print($error);
-        return false;
     }
 }

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

src/Libs/View.php

@@ -17,14 +17,14 @@ class View extends Neuron {
      * Incluye el 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.
      *
      * @return void
      */
     protected function include(
         string  $viewName,
-        string $viewPath  = null,
+        ?string $viewPath  = null,
         string  $extension = 'php'
     ): void
     {
@@ -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,26 +52,26 @@ 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 $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
      */
     public function html(
         string  $viewName,
-        string $viewPath  = null,
+        ?string $viewPath  = null,
         string  $extension = 'php'
     ): void
     {
@@ -86,14 +86,14 @@ 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 $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
      */
     public function css(
         string  $viewName,
-        string $viewPath  = null,
+        ?string $viewPath  = null,
         string  $extension = 'css'
     ): void
     {
@@ -105,14 +105,14 @@ 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 $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
      */
     public function js(
         string  $viewName,
-        string $viewPath  = null,
+        ?string $viewPath  = null,
         string  $extension = 'js'
     ): void
     {