Fix some deprecations (#5)

* Removed the TreeBuilder deprecation * Fixed the getExtendedTypes() deprecation * Fixed a code error * Fixed syntax issue
tgalopin · Apr 2, 2019 · aabcda7 · aabcda7 · TomaszGasior · Apr 5, 2019
1 parent fe9c3d1
commit aabcda7
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 4 deletions.
diff --git a/src/DependencyInjection/Configuration.php b/src/DependencyInjection/Configuration.php
@@ -23,9 +23,8 @@ class Configuration implements ConfigurationInterface
 {
     public function getConfigTreeBuilder()
     {
-        $treeBuilder = new TreeBuilder();
-
-        $rootNode = $treeBuilder->root('html_sanitizer');
+        $treeBuilder = new TreeBuilder('html_sanitizer');
+        $rootNode = $this->getRootNode($treeBuilder, 'html_sanitizer');
 
         $rootNode
             ->children()
@@ -40,4 +39,14 @@ public function getConfigTreeBuilder()
 
         return $treeBuilder;
     }
+
+    private function getRootNode(TreeBuilder $treeBuilder, $name)
+    {
+        // BC layer for symfony/config 4.1 and older
+        if (!\method_exists($treeBuilder, 'getRootNode')) {
+            return $treeBuilder->root($name);
+        }
+
+        return $treeBuilder->getRootNode();
+    }
 }
diff --git a/src/Form/TextTypeExtension.php b/src/Form/TextTypeExtension.php
@@ -33,9 +33,15 @@ public function __construct(ContainerInterface $sanitizers, string $default)
         $this->default = $default;
     }
 
+    // needed for BC reasons
     public function getExtendedType()
     {
-        return TextType::class;
+        return self::getExtendedTypes()[0];
+    }
+
+    public static function getExtendedTypes()
+    {
+        return [TextType::class];
     }
 
     public function configureOptions(OptionsResolver $resolver)