Skip to content

Commit 4acba1f

Browse files
committed
fix configuration if not using phpcr provider. fix #124
1 parent 6b3bab6 commit 4acba1f

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

DependencyInjection/CmfRoutingExtension.php

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,11 +87,12 @@ private function setupDynamicRouter(array $config, ContainerBuilder $container,
8787
$loader->load('dynamic_routing.xml');
8888

8989
$hasProvider = false;
90+
$hasContentRepository = false;
9091
if (!empty($config['phpcr_provider']['enabled'])) {
9192
$this->loadPhpcrProvider($config['phpcr_provider'], $loader, $container);
9293
$hasProvider = true;
94+
$hasContentRepository = true;
9395
}
94-
9596
if (isset($config['route_provider_service_id'])) {
9697
$container->setAlias('cmf_routing.route_provider', $config['route_provider_service_id']);
9798
$hasProvider = true;
@@ -101,6 +102,14 @@ private function setupDynamicRouter(array $config, ContainerBuilder $container,
101102
}
102103
if (isset($config['content_repository_service_id'])) {
103104
$container->setAlias('cmf_routing.content_repository', $config['content_repository_service_id']);
105+
$hasContentRepository = true;
106+
}
107+
// content repository is optional
108+
if ($hasContentRepository) {
109+
$generator = $container->getDefinition($this->getAlias() . '.generator');
110+
$generator->addMethodCall('setContentRepository', array(
111+
new Reference($this->getAlias() . '.content_repository'),
112+
));
104113
}
105114

106115
$dynamic = $container->getDefinition($this->getAlias().'.dynamic_router');

Resources/config/dynamic_routing.xml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,6 @@
7676
<argument type="service" id="cmf_routing.route_provider" />
7777
<argument type="service" id="logger" on-invalid="ignore" />
7878
<call method="setContainer"><argument type="service" id="service_container"/></call>
79-
<call method="setContentRepository"><argument type="service" id="cmf_routing.content_repository"/></call>
8079
</service>
8180

8281
<service id="cmf_routing.redirect_controller" class="%cmf_routing.redirect_controller_class%">

0 commit comments

Comments
 (0)