Skip to content

Commit 4e7df16

Browse files
SelviAOliboy50
andauthored
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 <Oliboy50@users.noreply.github.com>
1 parent a9f9f46 commit 4e7df16

2 files changed

Lines changed: 9 additions & 3 deletions

File tree

composer.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
},
2323
"require-dev": {
2424
"atoum/atoum": "^2.8|^3.0",
25-
"symfony/symfony": "^3.4 || ^4.0",
25+
"symfony/symfony": "^3.4 || ^4.0 || ^5.0",
2626
"m6web/coke" : "~1.2",
2727
"m6web/symfony2-coding-standard" : "~1.2"
2828
},

src/DependencyInjection/Configuration.php

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,8 +18,14 @@ class Configuration implements ConfigurationInterface
1818
*/
1919
public function getConfigTreeBuilder()
2020
{
21-
$treeBuilder = new TreeBuilder();
22-
$rootNode = $treeBuilder->root('m6_statsd');
21+
// symfony/config >= 4.2.0
22+
if (\method_exists(TreeBuilder::class, 'getRootNode')) {
23+
$treeBuilder = new TreeBuilder('m6_statsd');
24+
$rootNode = $treeBuilder->getRootNode();
25+
} else {
26+
$treeBuilder = new TreeBuilder();
27+
$rootNode = $treeBuilder->root('m6_statsd');
28+
}
2329

2430
$this->addServersSection($rootNode);
2531
$this->addClientsSection($rootNode);

0 commit comments

Comments
 (0)