Updated the default route matcher class with minor fix

divineniiquaye · Apr 14, 2022 · 5bc5bf6 · 5bc5bf6
1 parent d5e0aed
commit 5bc5bf6
Showing 1 changed file with 7 additions and 7 deletions.
diff --git a/src/RouteMatcher.php b/src/RouteMatcher.php
@@ -144,6 +144,13 @@ protected function matchCollection(string $method, UriInterface $uri, RouteColle
                 continue;
             }
 
+            [$pathRegex, $hostsRegex, $variables] = $this->optimized[$offset] ??= $this->compiler->compile($route);
+            $hostsVar = [];
+
+            if (!\preg_match($pathRegex, $requestPath, $matches, \PREG_UNMATCHED_AS_NULL)) {
+                continue;
+            }
+
             if (!$route->hasMethod($method)) {
                 $requirements[0] = \array_merge($requirements[0] ?? [], $route->getMethods());
                 continue;
@@ -154,13 +161,6 @@ protected function matchCollection(string $method, UriInterface $uri, RouteColle
                 continue;
             }
 
-            [$pathRegex, $hostsRegex, $variables] = $this->optimized[$offset] ??= $this->compiler->compile($route);
-            $hostsVar = [];
-
-            if (!\preg_match($pathRegex, $requestPath, $matches, \PREG_UNMATCHED_AS_NULL)) {
-                continue;
-            }
-
             if (empty($hostsRegex) || $this->matchHost($hostsRegex, $uri, $hostsVar)) {
                 if (!empty($variables)) {
                     $matchInt = 0;