Skip to content

Commit b35d6b9

Browse files
committed
check sense of auto_locale_pattern
1 parent f9f4896 commit b35d6b9

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

DependencyInjection/CmfRoutingExtension.php

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -94,8 +94,11 @@ private function setupDynamicRouter(array $config, ContainerBuilder $container,
9494
$container->setParameter($this->getAlias() . '.uri_filter_regexp', $config['uri_filter_regexp']);
9595
$container->setParameter($this->getAlias() . '.route_collection_limit', $config['route_collection_limit']);
9696

97-
$locales = !empty($config['locales']) ? $config['locales'] : array();
97+
$locales = empty($config['locales']) ? array() : $config['locales'];
9898
$container->setParameter($this->getAlias() . '.dynamic.locales', $locales);
99+
if (count($locales) === 0 && $config['auto_locale_pattern']) {
100+
throw new InvalidConfigurationException('It makes no sense to activate auto_locale_pattern when no locales are configured.');
101+
}
99102
$container->setParameter($this->getAlias() . '.dynamic.auto_locale_pattern', $config['auto_locale_pattern']);
100103

101104
$container->setParameter($this->getAlias() . '.dynamic.limit_candidates', $config['limit_candidates']);

0 commit comments

Comments
 (0)