Merge pull request #14 from skipperbent/development

skipperbent · skipperbent · commit 99da70874e78 · 2015-10-24T23:24:52.000+02:00
[BUGFIX] Fixed missing "/" in getRoute on some rare occasions.
diff --git a/src/Pecee/SimpleRouter/RouterBase.php b/src/Pecee/SimpleRouter/RouterBase.php
@@ -233,7 +233,7 @@ protected function processUrl($route, $method = null, $parameters = null, $getPa
             } else {
                 // If no parameters are specified in the route, assume that the provided parameters should be used.
                 if(count($parameters)) {
-                    $url .= join('/', $parameters);
+                    $url = rtrim($url, '/') . '/' . join('/', $parameters);
                 }
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -233,7 +233,7 @@ protected function processUrl($route, $method = null, $parameters = null, $getPa`
`233`	`233`	`} else {`
`234`	`234`	`// If no parameters are specified in the route, assume that the provided parameters should be used.`
`235`	`235`	`if(count($parameters)) {`
`236`		`- $url .= join('/', $parameters);`
	`236`	`+ $url = rtrim($url, '/') . '/' . join('/', $parameters);`
`237`	`237`	`}`
`238`	`238`	`}`
`239`	`239`	`}`