ivo/ponzi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fixed route handles for uri's that partially match

Browse Source
This commit is contained in:
Eric Barnes
2012-01-19 08:47:17 -05:00
parent 9bcbe6a357
commit ea8c6e1825
2 changed files with 3 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
laravel/routing/route.php
View File

@@ -214,7 +214,7 @@ class Route {
*/ */
public function handles($uri) public function handles($uri)
{ {
$pattern = ($uri !== '/') ? str_replace('*', '(.*)', $uri) : '^/$'; $pattern = ($uri !== '/') ? str_replace('*', '(.*)', $uri).'\z' : '^/$';
return ! is_null(array_first($this->uris, function($key, $uri) use ($pattern) return ! is_null(array_first($this->uris, function($key, $uri) use ($pattern)
{ {

1
tests/cases/laravel/route.test.php
View File

@@ -24,6 +24,7 @@ class RouteTest extends PHPUnit_Framework_TestCase {
$this->assertFalse($route->handles('/')); $this->assertFalse($route->handles('/'));
$this->assertFalse($route->handles('baz')); $this->assertFalse($route->handles('baz'));
$this->assertFalse($route->handles('/foo')); $this->assertFalse($route->handles('/foo'));
$this->assertFalse($route->handles('foo'));
$route = new Laravel\Routing\Route('GET /', array('handles' => array('GET /', 'GET /home'))); $route = new Laravel\Routing\Route('GET /', array('handles' => array('GET /', 'GET /home')));