[Serializer] Handle true and false appropriately in CSV encoder

battye · fabpot · commit 89cba00c6890 · 2019-06-14T07:49:57.000+02:00
diff --git a/src/Symfony/Component/Serializer/Encoder/CsvEncoder.php b/src/Symfony/Component/Serializer/Encoder/CsvEncoder.php
@@ -189,7 +189,8 @@ private function flatten(array $array, array &$result, $keySeparator, $parentKey
             if (\is_array($value)) {
                 $this->flatten($value, $result, $keySeparator, $parentKey.$key.$keySeparator);
             } else {
-                $result[$parentKey.$key] = $value;
+                // Ensures an actual value is used when dealing with true and false
+                $result[$parentKey.$key] = false === $value ? 0 : (true === $value ? 1 : $value);
             }
         }
     }
diff --git a/src/Symfony/Component/Serializer/Tests/Encoder/CsvEncoderTest.php b/src/Symfony/Component/Serializer/Tests/Encoder/CsvEncoderTest.php
@@ -29,6 +29,24 @@ protected function setUp()
         $this->encoder = new CsvEncoder();
     }
 
+    public function testTrueFalseValues()
+    {
+        $data = [
+            'string' => 'foo',
+            'int' => 2,
+            'false' => false,
+            'true' => true,
+        ];
+
+        // Check that true and false are appropriately handled
+        $this->assertEquals(<<<'CSV'
+string,int,false,true
+foo,2,0,1
+
+CSV
+    , $this->encoder->encode($data, 'csv'));
+    }
+
     public function testSupportEncoding()
     {
         $this->assertTrue($this->encoder->supportsEncoding('csv'));

Original file line number	Diff line number	Diff line change
`@@ -189,7 +189,8 @@ private function flatten(array $array, array &$result, $keySeparator, $parentKey`
`189`	`189`	`if (\is_array($value)) {`
`190`	`190`	`$this->flatten($value, $result, $keySeparator, $parentKey.$key.$keySeparator);`
`191`	`191`	`} else {`
`192`		`- $result[$parentKey.$key] = $value;`
	`192`	`+ // Ensures an actual value is used when dealing with true and false`
	`193`	`+ $result[$parentKey.$key] = false === $value ? 0 : (true === $value ? 1 : $value);`
`193`	`194`	`}`
`194`	`195`	`}`
`195`	`196`	`}`