Skip to content

Commit 4407bf6

Browse files
committed
sanity check configuration
1 parent 5ff3536 commit 4407bf6

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

DependencyInjection/CmfRoutingExtension.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
namespace Symfony\Cmf\Bundle\RoutingBundle\DependencyInjection;
44

5+
use Symfony\Component\Config\Definition\Exception\InvalidConfigurationException;
56
use Symfony\Component\DependencyInjection\ContainerBuilder;
67
use Symfony\Component\Config\FileLocator;
78
use Symfony\Component\Config\Definition\Processor;
@@ -85,12 +86,18 @@ private function setupDynamicRouter(array $config, ContainerBuilder $container,
8586

8687
$loader->load('dynamic_routing.xml');
8788

89+
$hasProvider = false;
8890
if (!empty($config['phpcr_provider']['enabled'])) {
8991
$this->loadPhpcrProvider($config['phpcr_provider'], $loader, $container);
92+
$hasProvider = true;
9093
}
9194

9295
if (isset($config['route_provider_service_id'])) {
9396
$container->setAlias('cmf_routing.route_provider', $config['route_provider_service_id']);
97+
$hasProvider = true;
98+
}
99+
if (!$hasProvider) {
100+
throw new InvalidConfigurationException('When the dynamic router is enabled, you need to either set dynamic.phpcr_provider.enabled or specify dynamic.route_provider_service_id');
94101
}
95102
if (isset($config['content_repository_service_id'])) {
96103
$container->setAlias('cmf_routing.content_repository', $config['content_repository_service_id']);

0 commit comments

Comments
 (0)