src/Libs/Neuron.php

@@ -22,7 +22,7 @@ class Neuron {
     
     private $data;
   
-    public function __construct($data = []){
+    public function __construct($data){
       $this->data = (array) $data;
     }
     

src/Libs/Router.php

@@ -19,10 +19,7 @@ class Router {
   private static $put = [];
   private static $delete = [];
   private static $last;
-  public static $notFoundCallBack = function () {
-    header("HTTP/1.0 404 Not Found");
-    echo '<h2 style="text-align: center;margin: 25px 0px;">Error 404 - Página no encontrada</h2>';
-  };
+  public static $notFoundCallBack;
 
   private function __construct() {}
 
@@ -41,9 +38,6 @@ class Router {
   *     callback    - Contiene el callback en formato Namespace\Clase::Método
   */
   private static function parse($path, $callback) {
-    preg_match_all('/{([\w-]+)}/s', $path, $matches, PREG_PATTERN_ORDER);
-    $paramNames = $matches[1];
-
     $path = preg_quote($path, '/');
     $path = preg_replace(
             ['/\\\{[\w-]+\\\}/s'],
@@ -53,11 +47,9 @@ class Router {
     if (!is_callable($callback)) {
       $callback = 'Controllers\\'.$callback;
     }
-
     return [
-      'path'       => $path,
-      'callback'   => $callback,
-      'paramNames' => $paramNames
+      'path'=> $path,
+      'callback' => $callback
     ];
   }
 
@@ -71,7 +63,7 @@ class Router {
   public static function baseURI() {
     if (defined('SITE_URL'))
       return parse_url(SITE_URL, PHP_URL_PATH);
-    return str_replace($_SERVER['DOCUMENT_ROOT'], '/', ROOT_DIR);
+    return str_replace($_SERVER['DOCUMENT_ROOT'],'/', ROOT_DIR);
   }
 
   /*
@@ -91,26 +83,14 @@ class Router {
   /*
   * Añade un middleware a la última ruta usada.
   * Solo se puede usar un middleware a la vez.
-  *
-  * @param string $callback
-  *
-  * @return static
-  *   Devuelve un enlace estático
   */
-  public static function middleware($callback){
-    if (!isset(static::$last))
-      return;
+  public static function middleware($middleware){
+    if (!isset(static::$last)) return;
 
     $method = static::$last[0];
     $index = static::$last[1];
 
-    if (!is_callable($callback)) {
-      $callback = 'Middlewares\\'.$callback;
-    }
-
-    static::$$method[$index]['middleware'] = $callback;
-
-    return new static();
+    static::$$method[$index]['middleware'] = 'Middlewares\\'.$middleware;
   }
 
   /*
@@ -126,7 +106,6 @@ class Router {
     $req->get        = new Neuron($_GET);
     $req->post       = new Neuron($_POST);
     $req->json       = new Neuron(static::get_json());
-    $req->params     = new Neuron();
     $req->path       = static::URIPath();
     return $req;
   }
@@ -229,19 +208,37 @@ class Router {
   *
   * Este método ha de ser llamado luego de que todos los routers hayan sido configurados.
   *
-  * En caso que la URI actual coincida con un router configurado, se comprueba si hay middleware; Si hay
-  * middleware, se enviará el callback y los datos de la petición como un Neuron. Caso contrario, se enviarán
-  * los datos directamente al callback.
+  * En caso que la URI actual coincida con un router configurado,
+  * pueden suecedor los siguientes casos:
   *
-  * Con middleware:
-  *                 $middleware($callback, $req)
+  * 1. Tiene pseudovariables por URI
   *
-  * Sin middleware:
-  *                 $callback($req)
+  *   1.1. Tiene middleware, por lo que se llama al middleware enviándole los datos
+  *        en el siguiente orden:
+  *           - Función callback del router
+  *           - Objeto que contiene $_POST, $_GET y el JSON recibido
+  *           - Todas las pseudovariables (en el mismo orden que están escritas)
   *
-  * $req es una instancia de Neuron que tiene los datos de la petición.
+  *   1.1. No tiene middleware, por lo que se llama a la función callback del router
+  *        enviándole los datos en el siguiente orden:
+  *           - Todas las pseudovariables (en el mismo orden que están escritas)
+  *           - Objeto que contiene $_POST, $_GET y el JSON recibido
   *
-  * Si no la uri no coincide con ninguna de las
+  * 2. No tiene pseudovariables por URI
+  *
+  *   2.1. Tiene middleware, por lo que se llama al middleware enviándole los datos
+  *        En el siguiente orden:
+  *           - Función callback del router
+  *           - Todas las pseudovariables (en el mismo orden que están escritas)
+  *
+  *   2.2. No tiene middleware, por lo que se llama a la función callback del router
+  *        enviándole solo ún parámetro:
+  *           - Objeto que contiene $_POST, $_GET y el JSON recibido
+  *
+  * Nota: Gracias a que se usa call_user_func_array, los callbacks no es necesario que reciban
+  *       la misma cantidad de parámetros que se les envía. De ese modo, pueden recibir solo
+  *       los parámetros que van a utilizar, con el detalle de que siempre se respeta el orden
+  *       especificado anteriormente a la hora de recibirlos.
   */
   public static function apply() {
     $uri =  static::URIPath();
@@ -262,24 +259,31 @@ class Router {
     }
 
     foreach ($routers as $router) { // revisa todos los routers para ver si coinciden con la URI actual
-      if (preg_match_all('/^'.$router['path'].'\/?$/si',$uri, $matches, PREG_PATTERN_ORDER)) {
+      if (preg_match_all('/^'.$router['path'].'\/?$/si',$uri, $matches)){
         unset($matches[0]);
         $args = static::getReq();
 
-        // Comprobando pseudo variables en URI
-        if (isset($matches[1])) {
-          foreach ($matches as $index => $match) {
-            $paramName = $router['paramNames'][$index-1];
-            $args->params->$paramName = urldecode($match[0]);
+        if (isset($matches[1])) { // Caso 1 - Con pseudovariables por URI
+          $params = [];
+          foreach ($matches as $match) {
+            if (!empty($match))
+              $params[] = "$match[0]";
           }
-        }
 
-        // Comprobando si hay middleware
-        if (isset($router['middleware'])) {
-          //$middleware = explode('::',$router['middleware']);
-          $data = call_user_func_array($router['middleware'], [$router['callback'], $args]);
-        } else {
-          $data = call_user_func_array($router['callback'], [$args]);
+          if (isset($router['middleware'])) { // Caso 1.1 - Con middleware
+            $middleware = explode('::',$router['middleware']);
+            $data = call_user_func_array($middleware, [$router['callback'], $args, $params]);
+          } else { // Caso 1.2 - Sin middleware
+            $params[] = $args;
+            $data = call_user_func_array($router['callback'], $params);
+          }
+        } else { // Caso 2 - Sin Pseudo variables por URI
+          if (isset($router['middleware'])) { // Caso 2.1 - Con middleware
+            $middleware = explode('::',$router['middleware']);
+            $data = call_user_func_array($middleware, [$router['callback'], $args]);
+          } else { // Caso 2.2 - Sin middleware
+            $data = call_user_func_array($router['callback'], [$args]);
+          }
         }
 
         if (isset($data)) {
@@ -291,8 +295,12 @@ class Router {
       }
     }
 
-    // Si no hay router que coincida llamamos a $notFoundCallBack
-    call_user_func_array(static::$notFoundCallBack, [$req]);
+    if (isset(static::$notFoundCallBack))
+      call_user_func_array(static::$notFoundCallBack, []);
+    else {
+      header("HTTP/1.0 404 Not Found"); // Si no hay router que coincida, se devuelve error 404
+      echo '<h2 style="text-align: center;margin: 25px 0px;">Error 404 - Página no encontrada</h2>';
+    }
   }
 }
 ?>