Merge pull request #694 from KnifeLemon/master

fix wildcard routes being skipped by fast path method lookup closes #693

Author: n0nag0n

flight/net/Router.php

@@ -152,7 +152,7 @@ public function map(string $pattern, $callback, bool $pass_route = false, string
             if (!isset($this->routesByMethod[$method])) {
                 $this->routesByMethod[$method] = [];
             }
-            $this->routesByMethod[$method][] = $route;
+            $this->routesByMethod[$method][count($this->routes) - 1] = $route;
         }
 
         return $route;
@@ -270,17 +270,13 @@ public function route(Request $request)
         }
 
         // Fast path: check method-specific routes first, then wildcard routes (only on first routing attempt)
-        $methodsToCheck = [$requestMethod, '*'];
-        foreach ($methodsToCheck as $method) {
-            if (isset($this->routesByMethod[$method])) {
-                foreach ($this->routesByMethod[$method] as $route) {
-                    if ($route->matchUrl($requestUrl, $this->caseSensitive)) {
-                        $this->executedRoute = $route;
-                        // Set iterator position to this route for potential next() calls
-                        $this->index = array_search($route, $this->routes, true);
-                        return $route;
-                    }
-                }
+        $candidates = ($this->routesByMethod[$requestMethod] ?? []) + ($this->routesByMethod['*'] ?? []);
+        ksort($candidates);
+        foreach ($candidates as $routeIndex => $route) {
+            if ($route->matchUrl($requestUrl, $this->caseSensitive)) {
+                $this->executedRoute = $route;
+                $this->index = $routeIndex;
+                return $route;
             }
         }
 

tests/RouterTest.php

@@ -780,4 +780,21 @@ public function testStripMultipleSlashesFromUrlAndStillMatch(): void
         $this->request->method = 'GET';
         $this->check('OK');
     }
+    
+    public function testWildcardPassthroughRouteBeforeSpecificGetRoute(): void
+    {
+        $this->router->map('/@par/[^\/]+/*', function (string $par): bool {
+            echo "Passthrough (Par = $par)";
+            return true;
+        });
+
+        $this->router->map('GET /[^\/]+/target', function (): void {
+            echo ' Target';
+        });
+
+        $this->request->url = '/foobar/target/';
+        $this->request->method = 'GET';
+
+        $this->check('Passthrough (Par = foobar) Target');
+    }
 }