add the getCause() to the ConstraintViolationInterface

xabbuh · xabbuh · commit 3914ce0ff4ab · 2022-09-29T19:55:49.000+02:00
diff --git a/src/Symfony/Component/Form/Extension/DataCollector/FormDataExtractor.php b/src/Symfony/Component/Form/Extension/DataCollector/FormDataExtractor.php
@@ -98,7 +98,13 @@ public function extractSubmittedData(FormInterface $form): array
             while (null !== $cause) {
                 if ($cause instanceof ConstraintViolationInterface) {
                     $errorData['trace'][] = $cause;
-                    $cause = method_exists($cause, 'getCause') ? $cause->getCause() : null;
+
+                    if (method_exists($cause, 'getCause')) {
+                        $cause = $cause->getCause();
+                    } else {
+                        trigger_deprecation('symfony/validator', '6.2', 'Not implementing the getCause() method in "%s" is deprecated.', $cause::class);
+                        $cause = null;
+                    }
 
                     continue;
                 }
diff --git a/src/Symfony/Component/Validator/CHANGELOG.md b/src/Symfony/Component/Validator/CHANGELOG.md
@@ -4,6 +4,7 @@ CHANGELOG
 6.2
 ---
 
+ * Add method `getCause()` to `ConstraintViolationInterface`
  * Add the `When` constraint and validator
  * Deprecate the "loose" e-mail validation mode, use "html5" instead
  * Add the `negate` option to the `Expression` constraint, to inverse the logic of the violation's creation
diff --git a/src/Symfony/Component/Validator/ConstraintViolationInterface.php b/src/Symfony/Component/Validator/ConstraintViolationInterface.php
@@ -31,6 +31,7 @@
  *
  * @author Bernhard Schussek <bschussek@gmail.com>
  *
+ * @method mixed getCause()    Returns the cause of the violation. Not implementing it is deprecated since Symfony 6.2.
  * @method string __toString() Converts the violation into a string for debugging purposes. Not implementing it is deprecated since Symfony 6.1.
  */
 interface ConstraintViolationInterface

Original file line number	Diff line number	Diff line change
`@@ -31,6 +31,7 @@`
`31`	`31`	`*`
`32`	`32`	`* @author Bernhard Schussek <[email protected]>`
`33`	`33`	`*`
	`34`	`+ * @method mixed getCause() Returns the cause of the violation. Not implementing it is deprecated since Symfony 6.2.`
`34`	`35`	`* @method string __toString() Converts the violation into a string for debugging purposes. Not implementing it is deprecated since Symfony 6.1.`
`35`	`36`	`*/`
`36`	`37`	`interface ConstraintViolationInterface`