Router gracefully falls back to default API version if no exact matching collection is found.

Signed-off-by: Jason Lewis <jason.lewis1991@gmail.com>

Author: jasonlewis

src/Routing/Router.php

@@ -449,7 +449,20 @@ protected function parseAcceptHeader($request)
 	 */
 	public function getApiRouteCollectionFromRequest(Request $request)
 	{
-		return array_first($this->api, function($k, $c) use ($request) { return $c->matchesRequest($request); }, null);
+		$collection = array_first($this->api, function($key, $collection) use ($request)
+		{
+			return $collection->matchesRequest($request);
+		});
+
+		// If we don't initially find a collection then we'll grab the default
+		// version collection instead. This is a sort of graceful fallback
+		// and allows viewing of the latest API version in the browser.
+		if ( ! $collection)
+		{
+			return $this->getApiRouteCollection($this->defaultVersion);
+		}
+
+		return $collection;
 	}
 
 	/**

tests/RoutingRouterTest.php

@@ -404,4 +404,25 @@ public function testApiCollectionsWithPointReleaseVersions()
 	}
 
 
+	public function testRouterDefaultsToDefaultVersionCollectionWhenNoAcceptHeader()
+	{
+		$this->router->api(['version' => 'v1', 'prefix' => 'api'], function()
+		{
+			$this->router->get('foo', function() { return 'bar'; });
+		});
+
+		$this->router->api(['version' => 'v2', 'prefix' => 'api'], function()
+		{
+			$this->router->get('foo', function() { return 'baz'; });
+		});
+
+		$request = Request::create('api/foo', 'GET');
+
+		$this->assertEquals('{"message":"bar"}', $this->router->dispatch($request)->getContent());
+
+		$this->router->setDefaultVersion('v2');
+		$this->assertEquals('{"message":"baz"}', $this->router->dispatch($request)->getContent());
+	}
+
+
 }