[Form] Preventing validation of children if parent with Valid constra…

…int has no validation groups
symfony · fabpot · Mar 20, 2019 · Mar 18, 2019 · Mar 19, 2019 · f45f0d03fc4adf347c487519b436c76bde95f8bd
commit f45f0d03fc4adf347c487519b436c76bde95f8bd
@@ -44,6 +44,11 @@ public function validate($form, Constraint $formConstraint)
        if ($form->isSubmitted() && $form->isSynchronized()) {
            // Validate the form data only if transformation succeeded
            $groups = self::getValidationGroups($form);
+
+            if (!$groups) {
+                return;
+            }
+
            $data = $form->getData();

            // Validate the data against its own constraints

@@ -220,6 +220,28 @@ public function testDontValidateConstraintsIfNoValidationGroups()
        $this->assertNoViolation();
    }

+    public function testDontValidateChildConstraintsIfCallableNoValidationGroups()
+    {
+        $formOptions = [
+            'constraints' => [new Valid()],
+            'validation_groups' => [],
+        ];
+        $form = $this->getBuilder('name', null, $formOptions)
+            ->setCompound(true)
+            ->setDataMapper(new PropertyPathMapper())
+            ->getForm();
+        $childOptions = ['constraints' => [new NotBlank()]];
+        $child = $this->getCompoundForm(new \stdClass(), $childOptions);
+        $form->add($child);
+        $form->submit([]);
+
+        $this->expectNoValidate();
+
+        $this->validator->validate($form, new Form());
+
+        $this->assertNoViolation();
+    }
+
    public function testDontValidateIfNotSynchronized()
    {
        $object = new \stdClass();