Merge pull request #258 from symfony-cmf/publish-workflow-auto

do not try to load the publish workflow service if the CmfCoreBundle is not loaded

Author: dbu

CHANGELOG.md

@@ -1,6 +1,12 @@
 Changelog
 =========
 
+unreleased
+----------
+
+* **2016-05-25** Use "auto" for publish workflow enabled flag. If auto and
+  CmfCoreBundle is not instantiated, publish workflow integration is not enabled.
+
 2.0.0
 -----
 

DependencyInjection/CmfMenuExtension.php

@@ -44,7 +44,9 @@ public function load(array $configs, ContainerBuilder $container)
             $this->loadExtensions($config, $loader, $container);
         }
 
-        if ($config['publish_workflow']['enabled']) {
+        if (true === $config['publish_workflow']['enabled']
+            || 'auto' === $config['publish_workflow']['enabled'] && isset($bundles['CmfCoreBundle'])
+        ) {
             $loader->load('publish-workflow.xml');
         }
     }

DependencyInjection/Configuration.php

@@ -81,7 +81,12 @@ public function getConfigTreeBuilder()
 
                 ->arrayNode('publish_workflow')
                     ->addDefaultsIfNotSet()
-                    ->canBeDisabled()
+                    ->children()
+                        ->enumNode('enabled')
+                            ->values(array(true, false, 'auto'))
+                            ->defaultValue('auto')
+                        ->end()
+                    ->end()
                 ->end()
             ->end()
         ;

Resources/config/schema/menu-1.0.xsd

@@ -51,7 +51,7 @@
     </xsd:complexType>
 
     <xsd:complexType name="publish-workflow">
-        <xsd:attribute name="enabled" type="xsd:boolean" />
+        <xsd:attribute name="enabled" type="enable_auto" />
     </xsd:complexType>
 
     <xsd:simpleType name="enable_auto">