Merge pull request #534 from flightphp/alias-bug

fixed bug with root alias

composer.json

@@ -58,7 +58,7 @@
         "test-coverage": "rm clover.xml && XDEBUG_MODE=coverage vendor/bin/phpunit --coverage-html=coverage --coverage-clover=clover.xml && vendor/bin/coverage-check clover.xml 100",
         "lint": "phpstan --no-progress -cphpstan.neon",
         "beautify": "phpcbf --standard=phpcs.xml",
-        "phpcs": "phpcs --standard=phpcs.xml"
+        "phpcs": "phpcs --standard=phpcs.xml -n"
     },
     "suggest": {
         "latte/latte": "Latte template engine",

flight/net/Route.php

@@ -192,7 +192,9 @@ class Route
         // catches potential optional parameter
         $url = str_replace('(/', '/', $url);
         // trim any trailing slashes
+        if ($url !== '/') {
             $url = rtrim($url, '/');
+        }
         return $url;
     }
 

tests/RouterTest.php

@@ -533,6 +533,13 @@ class RouterTest extends TestCase
         $this->assertTrue($result);
     }
 
+    public function testGetRootUrlByAlias()
+    {
+        $this->router->map('/', [$this, 'ok'], false, 'path1');
+        $url = $this->router->getUrlByAlias('path1');
+        $this->assertEquals('/', $url);
+    }
+
     public function testGetUrlByAliasNoMatches()
     {
         $this->router->map('/path1', [$this, 'ok'], false, 'path1');