Allow null as typed value for any not required field

Author: n0str

EditorJS/BlockHandler.php

@@ -136,6 +136,13 @@ private function validate($rules, $blockData)
                 continue;
             }
 
+            /**
+             * Do not check element type if it is not required and null
+             */
+            if (isset($rule['required']) && $rule['required'] === false && $value === null) {
+                continue;
+            }
+
             /**
              * Validate element types
              */

tests/TypeTest.php

@@ -11,6 +11,7 @@
 class TypeTest extends TestCase
 {
     const CONFIGURATION_FILE = TESTS_DIR . "/samples/type-test-config.json";
+    const CONFIGURATION_FILE_REQUIRED = TESTS_DIR . "/samples/type-test-config-required.json";
 
     /**
      * Sample configuration
@@ -66,4 +67,19 @@ public function testStringFailed()
 
         $this->assertException($callable, EditorJSException::class, null, 'Option \'string_test\' with value `17` must be string');
     }
+
+    public function testNullNotRequired()
+    {
+        new EditorJS('{"blocks":[{"type":"test","data":{"string_test": null}}]}', $this->configuration);
+    }
+
+    public function testNullRequired()
+    {
+        new EditorJS('{"blocks":[{"type":"test","data":{"string_test": "qwe"}}]}', file_get_contents(TypeTest::CONFIGURATION_FILE_REQUIRED));
+
+        $callable = function () {
+            new EditorJS('{"blocks":[{"type":"test","data":{"string_test": null}}]}', file_get_contents(TypeTest::CONFIGURATION_FILE_REQUIRED));
+        };
+        $this->assertException($callable, EditorJSException::class, null, 'Not found required param `string_test`');
+    }
 }

tests/samples/type-test-config-required.json

@@ -0,0 +1,10 @@
+{
+  "tools": {
+    "test": {
+      "string_test": {
+        "type": "string",
+        "required": true
+      }
+    }
+  }
+}