Commit 73c09f2f authored by Taylor Otwell's avatar Taylor Otwell

fixed bug in router.

parent 6865db3f
...@@ -62,7 +62,7 @@ class Auth { ...@@ -62,7 +62,7 @@ class Auth {
{ {
if ( ! is_null(static::$user)) return static::$user; if ( ! is_null(static::$user)) return static::$user;
$id = Session::get(Auth::user_key); $id = IoC::core('session')->get(Auth::user_key);
// To retrieve the user, we'll first attempt to use the "user" Closure // To retrieve the user, we'll first attempt to use the "user" Closure
// defined in the auth configuration file, passing in the ID. The user // defined in the auth configuration file, passing in the ID. The user
......
...@@ -79,7 +79,7 @@ class Route { ...@@ -79,7 +79,7 @@ class Route {
*/ */
protected static function extract($segment) protected static function extract($segment)
{ {
$uri = substr($segment, strpos($segment, ' ')); $uri = substr($segment, strpos($segment, ' ') + 1);
return ($uri !== '/') ? trim($uri, '/') : $uri; return ($uri !== '/') ? trim($uri, '/') : $uri;
} }
......
...@@ -200,6 +200,8 @@ class Router { ...@@ -200,6 +200,8 @@ class Router {
$action = array('uses' => Bundle::prefix($bundle).'home@index'); $action = array('uses' => Bundle::prefix($bundle).'home@index');
$action['handles'] = array($destination);
return new Route($method.' '.$uri, $action); return new Route($method.' '.$uri, $action);
} }
...@@ -243,6 +245,8 @@ class Router { ...@@ -243,6 +245,8 @@ class Router {
$action = array('uses' => $prefix.$controller.'@'.$method); $action = array('uses' => $prefix.$controller.'@'.$method);
$action['handles'] = array($destination);
return new Route($destination, $action, $segments); return new Route($destination, $action, $segments);
} }
} }
......
...@@ -17,13 +17,13 @@ class RouteTest extends PHPUnit_Framework_TestCase { ...@@ -17,13 +17,13 @@ class RouteTest extends PHPUnit_Framework_TestCase {
*/ */
public function testHandlesIndicatesIfTheRouteHandlesAGivenURI() public function testHandlesIndicatesIfTheRouteHandlesAGivenURI()
{ {
$route = new Laravel\Routing\Route('GET /', array('handles' => array('foo/bar'))); $route = new Laravel\Routing\Route('GET /', array('handles' => array('GET /foo/bar')));
$this->assertTrue($route->handles('foo/*')); $this->assertTrue($route->handles('foo/*'));
$this->assertTrue($route->handles('foo/bar')); $this->assertTrue($route->handles('foo/bar'));
$this->assertFalse($route->handles('baz')); $this->assertFalse($route->handles('baz'));
$route = new Laravel\Routing\Route('GET /', array('handles' => array('/', 'home'))); $route = new Laravel\Routing\Route('GET /', array('handles' => array('GET /', 'GET /home')));
$this->assertTrue($route->handles('/')); $this->assertTrue($route->handles('/'));
$this->assertTrue($route->handles('home')); $this->assertTrue($route->handles('home'));
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment