Small bugfix in routing

CHANGELOG.md

@@ -1,5 +1,11 @@
 # Parable Routing
 
+## 0.2.3
+
+_Bugfix_
+
+- Small fix in `Router`, where while replacing parameters in a url we're matching would not check if the original url part was actually a parameter.
+
 ## 0.2.2
 
 _Changes_

src/Router.php

@@ -146,6 +146,10 @@ protected function matchParametered(string $httpMethod, string $urlToMatch): ?Ro
             foreach ($providedValues as $key => $value) {
                 $parameter = $explodedRouteUrl[$key];
 
+                if (strpos($parameter, '{') === false) {
+                    continue;
+                }
+
                 $valuesWithParameterAsKey[trim($parameter, '{}')] = $value;
 
                 $explodedUrlToMatch[$key] = $parameter;

tests/RouterTest.php

@@ -395,4 +395,18 @@ public function dpSimpleUrls(): array
             ['/simple/'],
         ];
     }
+
+    public function testSimilarUrlsDoNotConflictAndParametersAreReplacedCorrectly()
+    {
+        $this->router->add(['GET'], 'route1', 'route1/{param}', function () {});
+        $this->router->add(['GET'], 'route2', 'route2/{param}', function () {});
+
+        $matched = $this->router->match('GET', 'route1/test');
+
+        self::assertSame('route1', $matched->getName());
+
+        $matched = $this->router->match('GET', 'route2/test');
+
+        self::assertSame('route2', $matched->getName());
+    }
 }