RoutingExtension: creates service 'routes' only when routes are defin…

…ed (BC break)

src/Bridges/ApplicationDI/RoutingExtension.php

@@ -43,10 +43,13 @@ public function __construct(bool $debugMode = false)
 
 	public function loadConfiguration()
 	{
+		if (!$this->config->routes) {
+			return;
+		}
+
 		$builder = $this->getContainerBuilder();
 
 		$router = $builder->addDefinition($this->prefix('router'))
-			->setType(Nette\Routing\Router::class)
 			->setFactory(Nette\Application\Routers\RouteList::class);
 
 		if ($this->config->routeClass) {
@@ -80,13 +83,20 @@ public function beforeCompile()
 				new Definitions\Statement(Nette\Bridges\ApplicationTracy\RoutingPanel::class),
 			]);
 		}
+
+		if (!$builder->getByType(Nette\Routing\Router::class)) {
+			$builder->addDefinition($this->prefix('router'))
+				->setFactory(Nette\Routing\SimpleRouter::class);
+		}
 	}
 
 
 	public function afterCompile(Nette\PhpGenerator\ClassType $class)
 	{
 		if ($this->config->cache) {
-			$method = $class->getMethod(Nette\DI\Container::getMethodName($this->prefix('router')));
+			$builder = $this->getContainerBuilder();
+			$def = $builder->getDefinitionByType(Nette\Routing\Router::class);
+			$method = $class->getMethod(Nette\DI\Container::getMethodName($def->getName()));
 			try {
 				$router = eval($method->getBody());
 				if ($router instanceof Nette\Application\Routers\RouteList) {