Route: action is mandatory when defined as 'Presenter:'

Author: dg

src/Application/Routers/Route.php

@@ -112,14 +112,14 @@ class Route extends Nette\Object implements Application\IRouter
 	public function __construct($mask, $metadata = [], $flags = 0)
 	{
 		if (is_string($metadata)) {
-			$a = strrpos($metadata, ':');
+			$a = strrpos($tmp = $metadata, ':');
 			if (!$a) {
 				throw new Nette\InvalidArgumentException("Second argument must be array or string in format Presenter:action, '$metadata' given.");
 			}
-			$metadata = [
-				self::PRESENTER_KEY => substr($metadata, 0, $a),
-				'action' => $a === strlen($metadata) - 1 ? NULL : substr($metadata, $a + 1),
-			];
+			$metadata = [self::PRESENTER_KEY => substr($tmp, 0, $a)];
+			if ($a < strlen($tmp) - 1) {
+				$metadata['action'] = substr($tmp, $a + 1);
+			}
 		} elseif ($metadata instanceof \Closure || $metadata instanceof Nette\Callback) {
 			if ($metadata instanceof Nette\Callback) {
 				trigger_error('Nette\Callback is deprecated, use Nette\Utils\Callback::toClosure().', E_USER_DEPRECATED);

tests/Routers/Route.mandatoryAction.phpt

@@ -19,3 +19,13 @@ testRouteIn($route, '/default', 'Default', [
 	'action' => 'default',
 	'test' => 'testvalue',
 ], '/default?test=testvalue');
+
+testRouteIn($route, '/', NULL);
+
+
+$route = new Route('<action>', 'Front:Default:');
+
+testRouteIn($route, '/default', 'Front:Default', [
+	'action' => 'default',
+	'test' => 'testvalue',
+], '/default?test=testvalue');