Skip to content

Commit cda5f93

Browse files
DZunkefabpot
authored andcommitted
[Form] Fix constraints could be null if not set
1 parent 211f89d commit cda5f93

File tree

2 files changed

+12
-1
lines changed

2 files changed

+12
-1
lines changed

Extension/Validator/Constraints/FormValidator.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ public function validate($form, Constraint $constraint)
4545

4646
// Validate the data against the constraints defined
4747
// in the form
48-
$constraints = $config->getOption('constraints');
48+
$constraints = $config->getOption('constraints', array());
4949
foreach ($constraints as $constraint) {
5050
foreach ($groups as $group) {
5151
if (in_array($group, $constraint->groups)) {

Tests/Extension/Validator/Constraints/FormValidatorTest.php

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,17 @@ public function testDontValidateIfParentWithoutCascadeValidation()
123123
$this->assertNoViolation();
124124
}
125125

126+
public function testNotExistingConstraintIndex()
127+
{
128+
$object = new \stdClass();
129+
$form = new FormBuilder('name', '\stdClass', $this->dispatcher, $this->factory);
130+
$form = $form->setData($object)->getForm();
131+
132+
$this->validator->validate($form, new Form());
133+
134+
$this->assertNoViolation();
135+
}
136+
126137
public function testValidateConstraintsEvenIfNoCascadeValidation()
127138
{
128139
$object = $this->getMock('\stdClass');

0 commit comments

Comments
 (0)