[Validator] Add object handling of invalid constraints in Composite

SenseException · fabpot · commit 4bd2c2287153 · 2017-03-22T12:05:13.000-07:00
diff --git a/src/Symfony/Component/Validator/Constraints/Composite.php b/src/Symfony/Component/Validator/Constraints/Composite.php
@@ -67,6 +67,10 @@ public function __construct($options = null)
 
         foreach ($nestedConstraints as $constraint) {
             if (!$constraint instanceof Constraint) {
+                if (is_object($constraint)) {
+                    $constraint = get_class($constraint);
+                }
+
                 throw new ConstraintDefinitionException(sprintf('The value %s is not an instance of Constraint in constraint %s', $constraint, get_class($this)));
             }
 
diff --git a/src/Symfony/Component/Validator/Tests/Constraints/CompositeTest.php b/src/Symfony/Component/Validator/Tests/Constraints/CompositeTest.php
@@ -125,6 +125,17 @@ public function testFailIfNoConstraint()
         ));
     }
 
+    /**
+     * @expectedException \Symfony\Component\Validator\Exception\ConstraintDefinitionException
+     */
+    public function testFailIfNoConstraintObject()
+    {
+        new ConcreteComposite(array(
+            new NotNull(array('groups' => 'Default')),
+            new \ArrayObject(),
+        ));
+    }
+
     /**
      * @expectedException \Symfony\Component\Validator\Exception\ConstraintDefinitionException
      */

Original file line number	Diff line number	Diff line change
`@@ -67,6 +67,10 @@ public function __construct($options = null)`
`67`	`67`
`68`	`68`	`foreach ($nestedConstraints as $constraint) {`
`69`	`69`	`if (!$constraint instanceof Constraint) {`
	`70`	`+ if (is_object($constraint)) {`
	`71`	`+ $constraint = get_class($constraint);`
	`72`	`+ }`
	`73`	`+`
`70`	`74`	`throw new ConstraintDefinitionException(sprintf('The value %s is not an instance of Constraint in constraint %s', $constraint, get_class($this)));`
`71`	`75`	`}`
`72`	`76`