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, ':');
-			if (!$a) {
+			$parts = explode(':', $metadata);
+			if (count($parts) !== 2) {
 				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 => $parts[0]];
+			if ($parts[1] !== '') {
+				$metadata['action'] = $parts[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,5 @@ testRouteIn($route, '/default', 'Default', [
 	'action' => 'default',
 	'test' => 'testvalue',
 ], '/default?test=testvalue');
+
+testRouteIn($route, '/', NULL);