Find fix for Collection and All validador

Benjamin Lazarecki · Benjamin Lazarecki · commit c1311b88de69 · 2016-02-06T15:44:28.000+01:00
diff --git a/src/Symfony/Component/Validator/Constraints/AllValidator.php b/src/Symfony/Component/Validator/Constraints/AllValidator.php
@@ -49,7 +49,7 @@ public function validate($value, Constraint $constraint)
         } else {
             // 2.4 API
             foreach ($value as $key => $element) {
-                $context->validateValue($element, $constraint->constraints, '['.$key.']');
+                $context->validateValue($element, $constraint->constraints, '['.$key.']', $context->getGroup());
             }
         }
     }
diff --git a/src/Symfony/Component/Validator/Constraints/CollectionValidator.php b/src/Symfony/Component/Validator/Constraints/CollectionValidator.php
@@ -63,7 +63,7 @@ public function validate($value, Constraint $constraint)
                             ->validate($value[$field], $fieldConstraint->constraints, $fieldConstraint->groups);
                     } else {
                         // 2.4 API
-                        $context->validateValue($value[$field], $fieldConstraint->constraints, '['.$field.']');
+                        $context->validateValue($value[$field], $fieldConstraint->constraints, '['.$field.']', $context->getGroup());
                     }
                 }
             } elseif (!$fieldConstraint instanceof Optional && !$constraint->allowMissingFields) {
diff --git a/src/Symfony/Component/Validator/ExecutionContext.php b/src/Symfony/Component/Validator/ExecutionContext.php
@@ -271,7 +271,15 @@ public function getMetadataFactory()
      */
     private function executeConstraintValidators($value, array $constraints)
     {
+        // Filter constraints with the context group.
+        $filteredConstraints = array();
         foreach ($constraints as $constraint) {
+            if (in_array($this->getGroup(), $constraint->groups)) {
+                $filteredConstraints[] = $constraint;
+            }
+        }
+
+        foreach ($filteredConstraints as $constraint) {
             $validator = $this->globalContext->getValidatorFactory()->getInstance($constraint);
             $validator->initialize($this);
             $validator->validate($value, $constraint);

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ public function validate($value, Constraint $constraint)`
`49`	`49`	`} else {`
`50`	`50`	`// 2.4 API`
`51`	`51`	`foreach ($value as $key => $element) {`
`52`		`- $context->validateValue($element, $constraint->constraints, '['.$key.']');`
	`52`	`+ $context->validateValue($element, $constraint->constraints, '['.$key.']', $context->getGroup());`
`53`	`53`	`}`
`54`	`54`	`}`
`55`	`55`	`}`
Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,7 @@ public function validate($value, Constraint $constraint)`
`63`	`63`	`->validate($value[$field], $fieldConstraint->constraints, $fieldConstraint->groups);`
`64`	`64`	`} else {`
`65`	`65`	`// 2.4 API`
`66`		`- $context->validateValue($value[$field], $fieldConstraint->constraints, '['.$field.']');`
	`66`	`+ $context->validateValue($value[$field], $fieldConstraint->constraints, '['.$field.']', $context->getGroup());`
`67`	`67`	`}`
`68`	`68`	`}`
`69`	`69`	`} elseif (!$fieldConstraint instanceof Optional && !$constraint->allowMissingFields) {`