fix: TreeBuilder (#72)

* fix: SF5 compatibility * refactor: check for getRootNode method existence * fix: test if getRootNode exits * fix(review): remove unused use statement Co-authored-by: Oliver THEBAULT <[email protected]>
BedrockStreaming · Jul 10, 2020 · 4e7df16 · 4e7df16
1 parent a9f9f46
commit 4e7df16
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/composer.json b/composer.json
@@ -22,7 +22,7 @@
     },
     "require-dev": {
         "atoum/atoum": "^2.8|^3.0",
-        "symfony/symfony": "^3.4 || ^4.0",
+        "symfony/symfony": "^3.4 || ^4.0 || ^5.0",
         "m6web/coke" : "~1.2",
         "m6web/symfony2-coding-standard" : "~1.2"
     },

diff --git a/src/DependencyInjection/Configuration.php b/src/DependencyInjection/Configuration.php
@@ -18,8 +18,14 @@ class Configuration implements ConfigurationInterface
      */
     public function getConfigTreeBuilder()
     {
-        $treeBuilder = new TreeBuilder();
-        $rootNode = $treeBuilder->root('m6_statsd');
+        // symfony/config >= 4.2.0
+        if (\method_exists(TreeBuilder::class, 'getRootNode')) {
+            $treeBuilder = new TreeBuilder('m6_statsd');
+            $rootNode = $treeBuilder->getRootNode();
+        } else {
+            $treeBuilder = new TreeBuilder();
+            $rootNode = $treeBuilder->root('m6_statsd');
+        }
 
         $this->addServersSection($rootNode);
         $this->addClientsSection($rootNode);