not use reflection to get value of FormType::$emptyData

peter-gribanov · peter-gribanov · commit 31f105e7b729 · 2021-06-07T19:05:57.000+03:00
diff --git a/src/Symfony/Component/Form/Extension/Core/Type/FormType.php b/src/Symfony/Component/Form/Extension/Core/Type/FormType.php
@@ -29,7 +29,12 @@
 class FormType extends BaseType
 {
     private $dataMapper;
-    private static $emptyData = null;
+
+    /**
+     * @internal
+     * @deprecated Don't use this property. It will be removed in symfony/form 6.0
+     */
+    public static $emptyData = null;
 
     public function __construct(PropertyAccessorInterface $propertyAccessor = null)
     {
@@ -159,7 +164,7 @@ public function configureOptions(OptionsResolver $resolver)
         };
 
         // Derive "empty_data" closure from "data_class" option
-        if (self::$emptyData === null) {
+        if (null === self::$emptyData) {
             self::$emptyData = function (Options $options) {
                 $class = $options['data_class'];
 
diff --git a/src/Symfony/Component/Form/Extension/Core/Type/TextType.php b/src/Symfony/Component/Form/Extension/Core/Type/TextType.php
@@ -27,13 +27,9 @@ public function buildForm(FormBuilderInterface $builder, array $options)
         // See https://github.com/symfony/symfony/issues/5906#issuecomment-203189375
         if ('' === $options['empty_data']) {
             $builder->addViewTransformer($this);
-        } elseif ($options['empty_data'] instanceof \Closure) {
-            $emptyData = new \ReflectionProperty(FormType::class, 'emptyData');
-            $emptyData->setAccessible(true);
-            // Closure in "empty_data" option is the default value from FormType::configureOptions()
-            if ($emptyData->getValue($builder->getForm()) === $options['empty_data']) {
-                trigger_deprecation('symfony/form', '5.4', 'The default value of "empty_data" option in "%s" will be changed to empty string. Declare "NULL" as value for "empty_data" if you still want use "NULL" as data.', __CLASS__);
-            }
+        } elseif (FormType::$emptyData === $options['empty_data']) {
+            // "empty_data" option is the default value from FormType::configureOptions()
+            trigger_deprecation('symfony/form', '5.4', 'The default value of "empty_data" option in "%s" will be changed to empty string. Declare "NULL" as value for "empty_data" if you still want use "NULL" as data.', __CLASS__);
         }
     }
 
