Merge pull request #5 from biigle/patch-1

Make parser more robust against invalid JSON

Author: mzur

src/Ifdo.php

@@ -84,27 +84,21 @@ public function getJsonData()
     public function getImageSetHeader()
     {
         $arr = $this->getJsonData();
-        if (array_key_exists('image-set-header', $arr))
-        {
+        if (is_array($arr) && array_key_exists('image-set-header', $arr)) {
             return $arr['image-set-header'];
         }
-        else
-        {
-            return [];
-        }
+
+        return [];
     }
 
     public function getImageSetItems()
     {
         $arr = $this->getJsonData();
-        if (array_key_exists('image-set-items', $arr))
-        {
+        if (is_array($arr) && array_key_exists('image-set-items', $arr)) {
             return $arr['image-set-items'];
         }
-        else
-        {
-            return [];
-        }
+
+        return [];
     }
 
     public function getIfdoVersion(): String

tests/IfdoTest.php

@@ -35,4 +35,12 @@ public function testStrictMode()
         $this->expectException(\Exception::class);
         Ifdo::fromString('{"some": "json"}', true);
     }
+
+    public function testInvalidJson()
+    {
+        $obj = Ifdo::fromString('{"some": "json",}');
+        $this->assertSame(null, $obj->getJsonData());
+        $this->assertSame([], $obj->getImageSetHeader());
+        $this->assertSame([], $obj->getImageSetItems());
+    }
 }