Merge pull request #26 from codex-editor/fix-array

n0str · web-flow · commit 6d0d6f1eaedb · 2018-10-14T17:57:29.000+03:00
Fix undefined key
diff --git a/EditorJS/BlockHandler.php b/EditorJS/BlockHandler.php
@@ -189,7 +189,15 @@ private function sanitize($rules, $blockData)
          * Sanitize every key in data block
          */
         foreach ($blockData as $key => $value) {
-            $rule = $rules[$key];
+            /**
+             * PHP Array has integer keys
+             */
+            if (is_integer($key)) {
+                $rule = $rules[BlockHandler::DEFAULT_ARRAY_KEY];
+            } else {
+                $rule = $rules[$key];
+            }
+
             $elementType = $rule['type'];
 
             /**
diff --git a/tests/BlockHandlerTest.php b/tests/BlockHandlerTest.php
@@ -59,4 +59,16 @@ public function testCanBeOnly()
 
         $this->assertException($callable, EditorJSException::class, null, 'Option \'level\' with value `5` has invalid value. Check canBeOnly param.');
     }
+
+    public function testListTool()
+    {
+        $data = '{"time":1539180803359,"blocks":[{"type":"list","data":{"style":"ordered","items":["first","second","third"]}}],"version":"2.1.1"}';
+        $editor = new EditorJS($data, $this->configuration);
+        $result = $editor->getBlocks();
+
+        $this->assertEquals(3, count($result[0]['data']['items']));
+        $this->assertEquals("first", $result[0]['data']['items'][0]);
+        $this->assertEquals("second", $result[0]['data']['items'][1]);
+        $this->assertEquals("third", $result[0]['data']['items'][2]);
+    }
 }
