Merge pull request #2 from martinsustek/json2csv-data-unification

Json2csv data unification

Author: ozdemirburak

src/File/Json.php

@@ -16,9 +16,28 @@ class Json extends AbstractFile
      */
     public function convert() : string
     {
-        return $this->toCsvString(array_map(function ($d) {
+        $data_decoded = json_decode($this->data, true);
+        $data_flattened = array_map(function ($d) {
             return $this->flatten($d);
-        }, json_decode($this->data, true)));
+        }, $data_decoded);
+
+        $keys = [];
+        foreach ($data_flattened as $entry_flattened) {
+            foreach ($entry_flattened as $key => $value) {
+                $keys[$key] = true;
+            }
+        }
+
+        $data_unified = [];
+        foreach ($data_flattened as $entry_flattened) {
+            $entry_unified = [];
+            foreach ($keys as $key => $foo) {
+                $entry_unified[$key] = array_key_exists($key, $entry_flattened) ? $entry_flattened[$key] : null;
+            }
+            $data_unified[] = $entry_unified;
+        }
+
+        return $this->toCsvString($data_unified);
     }
 
     /**

tests/JsonTest.php

@@ -45,6 +45,17 @@ public function testConversionAndDownload()
         $this->expectOutputRegex('/Turkey,"Republic of Turkey",Türkiye,783562,39,35\\n/');
     }
 
+    /**
+     * @group json-conversion-test
+     */
+    public function testConversionDynamicFormat()
+    {
+        $json = new Json(__DIR__ . '/data/dynamicFormat.json');
+        $result = ($json->convert());
+        $this->assertEquals(file_get_contents(__DIR__ . '/data/dynamicFormatResult.csv'), $result);
+    }
+
+
     /**
      * @return \OzdemirBurak\JsonCsv\File\Json
      */

tests/data/dynamicFormat.json

@@ -0,0 +1,10 @@
+[
+    {
+      "x": 0,
+      "y": 1
+    },
+    {
+      "y": 0,
+      "z": 0
+    }
+]

tests/data/dynamicFormatResult.csv

@@ -0,0 +1,3 @@
+x,y,z
+0,1,
+,0,0