Remove all "nullable-by-default-value" setters

Author: nicolas-grekas

Button.php

@@ -83,6 +83,9 @@ public function offsetUnset(mixed $offset): void
 
     public function setParent(FormInterface $parent = null): static
     {
+        if (1 > \func_num_args()) {
+            trigger_deprecation('symfony/form', '6.2', 'Calling "%s()" without any arguments is deprecated, pass null explicitly instead.', __METHOD__);
+        }
         if ($this->submitted) {
             throw new AlreadySubmittedException('You cannot set the parent of a submitted button.');
         }

ButtonBuilder.php

@@ -194,6 +194,10 @@ public function setAttributes(array $attributes): static
      */
     public function setDataMapper(DataMapperInterface $dataMapper = null): static
     {
+        if (1 > \func_num_args()) {
+            trigger_deprecation('symfony/form', '6.2', 'Calling "%s()" without any arguments is deprecated, pass null explicitly instead.', __METHOD__);
+        }
+
         throw new BadMethodCallException('Buttons do not support data mappers.');
     }
 

CHANGELOG.md

@@ -4,8 +4,11 @@ CHANGELOG
 6.2
 ---
 
-* Allow passing `TranslatableInterface` objects to the `ChoiceView` label
-* Allow passing `TranslatableInterface` objects to the `help` option
+ * Allow passing `TranslatableInterface` objects to the `ChoiceView` label
+ * Allow passing `TranslatableInterface` objects to the `help` option
+ * Deprecate calling `Button/Form::setParent()`, `ButtonBuilder/FormConfigBuilder::setDataMapper()`, `TransformationFailedException::setInvalidMessage()` without arguments
+ * Change the signature of `FormConfigBuilderInterface::setDataMapper()` to `setDataMapper(?DataMapperInterface)`
+ * Change the signature of `FormInterface::setParent()` to `setParent(?self)`
 
 6.1
 ---

Exception/TransformationFailedException.php

@@ -36,6 +36,9 @@ public function __construct(string $message = '', int $code = 0, \Throwable $pre
      */
     public function setInvalidMessage(string $invalidMessage = null, array $invalidMessageParameters = []): void
     {
+        if (1 > \func_num_args()) {
+            trigger_deprecation('symfony/form', '6.2', 'Calling "%s()" without any arguments is deprecated, pass null explicitly instead.', __METHOD__);
+        }
         $this->invalidMessage = $invalidMessage;
         $this->invalidMessageParameters = $invalidMessageParameters;
     }

Form.php

@@ -220,6 +220,10 @@ public function isDisabled(): bool
 
     public function setParent(FormInterface $parent = null): static
     {
+        if (1 > \func_num_args()) {
+            trigger_deprecation('symfony/form', '6.2', 'Calling "%s()" without any arguments is deprecated, pass null explicitly instead.', __METHOD__);
+        }
+
         if ($this->submitted) {
             throw new AlreadySubmittedException('You cannot set the parent of a submitted form.');
         }

FormConfigBuilder.php

@@ -340,6 +340,9 @@ public function setAttributes(array $attributes): static
 
     public function setDataMapper(DataMapperInterface $dataMapper = null): static
     {
+        if (1 > \func_num_args()) {
+            trigger_deprecation('symfony/form', '6.2', 'Calling "%s()" without any arguments is deprecated, pass null explicitly instead.', __METHOD__);
+        }
         if ($this->locked) {
             throw new BadMethodCallException('FormConfigBuilder methods cannot be accessed anymore once the builder is turned into a FormConfigInterface instance.');
         }

FormConfigBuilderInterface.php

@@ -100,7 +100,7 @@ public function setAttributes(array $attributes): static;
      *
      * @return $this
      */
-    public function setDataMapper(DataMapperInterface $dataMapper = null): static;
+    public function setDataMapper(?DataMapperInterface $dataMapper): static;
 
     /**
      * Sets whether the form is disabled.

FormInterface.php

@@ -34,7 +34,7 @@ interface FormInterface extends \ArrayAccess, \Traversable, \Countable
      * @throws Exception\LogicException            when trying to set a parent for a form with
      *                                             an empty name
      */
-    public function setParent(self $parent = null): static;
+    public function setParent(?self $parent): static;
 
     /**
      * Returns the parent form.