Added default 'auto' for enable_initializer configuration option

Author: pamil

DependencyInjection/CmfRoutingExtension.php

@@ -225,7 +225,7 @@ private function loadPhpcrProvider($config, XmlFileLoader $loader, ContainerBuil
         }
 
         if ($config['enable_initializer']) {
-            $loader->load('initializer-phpcr.xml');
+            $this->loadInitializer($config, $loader, $container);
         }
     }
 
@@ -242,6 +242,36 @@ private function loadSonataPhpcrAdmin($config, XmlFileLoader $loader, ContainerB
         $container->setParameter('cmf_routing.dynamic.persistence.phpcr.admin_basepath', $basePath);
     }
 
+    /**
+     * @param array $config
+     * @param XmlFileLoader $loader
+     * @param ContainerBuilder $container
+     */
+    private function loadInitializer($config, XmlFileLoader $loader, ContainerBuilder $container)
+    {
+        $adminBasepathParameter = $this->getAlias() . '.dynamic.persistence.phpcr.admin_basepath';
+
+        $initializedBasepaths = array();
+        if ($container->hasParameter($adminBasepathParameter)) {
+            $initializedBasepaths = array($container->getParameter($adminBasepathParameter));
+        }
+
+        if ('auto' === $config['enable_initializer'] && empty($initializedBasepaths)) {
+            return;
+        }
+
+        if (true === $config['enable_initializer'] && empty($initializedBasepaths)) {
+            $initializedBasepaths = $container->getParameter($this->getAlias() . '.dynamic.persistence.phpcr.route_basepaths');
+        }
+
+        $container->setParameter(
+            $this->getAlias() . '.dynamic.persistence.phpcr.initialized_basepaths',
+            $initializedBasepaths
+        );
+
+        $loader->load('initializer-phpcr.xml');
+    }
+
     private function loadOrmProvider($config, XmlFileLoader $loader, ContainerBuilder $container, $matchImplicitLocale)
     {
         $loader->load('provider-orm.xml');

DependencyInjection/Configuration.php

@@ -142,8 +142,26 @@ private function addDynamicSection(ArrayNodeDefinition $root)
                                             ->end()
                                             ->values(array(true, false, 'auto'))
                                             ->defaultValue('auto')
-                                        ->end() // use_sonata_admin
-                                        ->booleanNode('enable_initializer')->defaultTrue()->end()
+                                        ->end()
+                                        ->enumNode('enable_initializer')
+                                            ->beforeNormalization()
+                                                ->ifString()
+                                                ->then(function ($v) {
+                                                    switch ($v) {
+                                                        case 'true':
+                                                            return true;
+
+                                                        case 'false':
+                                                            return false;
+
+                                                        default:
+                                                            return $v;
+                                                    }
+                                                })
+                                            ->end()
+                                            ->values(array(true, false, 'auto'))
+                                            ->defaultValue('auto')
+                                        ->end()
                                     ->end()
                                 ->end() // phpcr
                                 ->arrayNode('orm')

Resources/config/initializer-phpcr.xml

@@ -8,9 +8,7 @@
 
         <service id="cmf_routing.initializer" class="Doctrine\Bundle\PHPCRBundle\Initializer\GenericInitializer">
             <argument>CmfRoutingBundle</argument>
-            <argument type="collection">
-                <argument>%cmf_routing.dynamic.persistence.phpcr.route_basepath%</argument>
-            </argument>
+            <argument>%cmf_routing.dynamic.persistence.phpcr.initialized_basepaths%</argument>
             <tag name="doctrine_phpcr.initializer"/>
         </service>
 

Tests/Unit/DependencyInjection/CmfRoutingExtensionTest.php

@@ -335,4 +335,48 @@ public function testInitializerEnabledEvenWithoutSonata()
 
         $this->assertTrue($this->container->has('cmf_routing.initializer'));
     }
+
+    public function testInitializerEnabledAutomaticallyIfSonataIsEnabled()
+    {
+        $this->container->setParameter(
+            'kernel.bundles',
+            array(
+                'CmfRoutingBundle' => true,
+                'SonataDoctrinePHPCRAdminBundle' => true,
+            )
+        );
+
+        $this->load(array(
+            'dynamic' => array(
+                'enabled' => true,
+                'persistence' => array(
+                    'phpcr' => array(
+                        'enabled' => true,
+                        'use_sonata_admin' => true,
+                        'enable_initializer' => 'auto',
+                    ),
+                ),
+            ),
+        ));
+
+        $this->assertTrue($this->container->has('cmf_routing.initializer'));
+    }
+
+    public function testInitializerDisabledAutomaticallyIfSonataIsDisabled()
+    {
+        $this->load(array(
+            'dynamic' => array(
+                'enabled' => true,
+                'persistence' => array(
+                    'phpcr' => array(
+                        'enabled' => true,
+                        'use_sonata_admin' => false,
+                        'enable_initializer' => 'auto',
+                    ),
+                ),
+            ),
+        ));
+
+        $this->assertFalse($this->container->has('cmf_routing.initializer'));
+    }
 }