Allow integer values in ChoiceType

mcfedr · mcfedr · commit dad87f8b3534 · 2017-03-10T11:50:31.000+02:00
diff --git a/src/Symfony/Component/Form/Extension/Core/Type/ChoiceType.php b/src/Symfony/Component/Form/Extension/Core/Type/ChoiceType.php
@@ -171,8 +171,8 @@ public function buildForm(FormBuilderInterface $builder, array $options)
             }
 
             foreach ($data as $v) {
-                if (null !== $v && !is_string($v)) {
-                    throw new TransformationFailedException('All choices submitted must be NULL or strings.');
+                if (null !== $v && !is_string($v) && !is_int($v)) {
+                    throw new TransformationFailedException('All choices submitted must be NULL, strings or ints.');
                 }
             }
         }, 256);
diff --git a/src/Symfony/Component/Form/Tests/Extension/Core/Type/ChoiceTypeTest.php b/src/Symfony/Component/Form/Tests/Extension/Core/Type/ChoiceTypeTest.php
@@ -2319,7 +2319,7 @@ public function testSubmitInvalidNestedValue($multiple, $expanded, $submissionDa
 
         $form->submit($submissionData);
         $this->assertFalse($form->isSynchronized());
-        $this->assertEquals('All choices submitted must be NULL or strings.', $form->getTransformationFailure()->getMessage());
+        $this->assertEquals('All choices submitted must be NULL, strings or ints.', $form->getTransformationFailure()->getMessage());
     }
 
     public function invalidNestedValueTestMatrix()

Original file line number	Diff line number	Diff line change
`@@ -171,8 +171,8 @@ public function buildForm(FormBuilderInterface $builder, array $options)`
`171`	`171`	`}`
`172`	`172`
`173`	`173`	`foreach ($data as $v) {`
`174`		`- if (null !== $v && !is_string($v)) {`
`175`		`- throw new TransformationFailedException('All choices submitted must be NULL or strings.');`
	`174`	`+ if (null !== $v && !is_string($v) && !is_int($v)) {`
	`175`	`+ throw new TransformationFailedException('All choices submitted must be NULL, strings or ints.');`
`176`	`176`	`}`
`177`	`177`	`}`
`178`	`178`	`}, 256);`
Original file line number	Diff line number	Diff line change
`@@ -2319,7 +2319,7 @@ public function testSubmitInvalidNestedValue($multiple, $expanded, $submissionDa`
`2319`	`2319`
`2320`	`2320`	`$form->submit($submissionData);`
`2321`	`2321`	`$this->assertFalse($form->isSynchronized());`
`2322`		`- $this->assertEquals('All choices submitted must be NULL or strings.', $form->getTransformationFailure()->getMessage());`
	`2322`	`+ $this->assertEquals('All choices submitted must be NULL, strings or ints.', $form->getTransformationFailure()->getMessage());`
`2323`	`2323`	`}`
`2324`	`2324`
`2325`	`2325`	`public function invalidNestedValueTestMatrix()`