clear errors from "additionalProperties: false"

CHANGELOG.md

@@ -24,6 +24,8 @@ should change the heading of the (upcoming) version to include a major version b
 - fixed an issue where all semantic props overwritten when a single [semantic theme-specific prop](https://react-jsonschema-form.readthedocs.io/en/latest/api-reference/themes/semantic-ui/uiSchema/) is passed in ([issue 2619](https://github.com/rjsf-team/react-jsonschema-form/issues/2619)) (https://github.com/rjsf-team/react-jsonschema-form/pull/2590)
 
 # v3.2.2 (upcoming)
+## @rjsf/core
+- clear errors on formData change when liveOmit=true when "additionalProperties: false" [issue 1507](https://github.com/rjsf-team/react-jsonschema-form/issues/1507) (https://github.com/rjsf-team/react-jsonschema-form/pull/2631)
 
 # v3.2.1
 

packages/core/package.json

@@ -16,8 +16,7 @@
     "start": "concurrently \"npm:build:* -- --watch\"",
     "tdd": "cross-env NODE_ENV=test mocha --require @babel/register --watch --require ./test/setup-jsdom.js test/**/*_test.js",
     "test": "cross-env BABEL_ENV=test NODE_ENV=test mocha --require @babel/register --require ./test/setup-jsdom.js test/**/*_test.js",
-    "test-coverage": "cross-env NODE_ENV=test nyc --reporter=lcov mocha --require @babel/register --require ./test/setup-jsdom.js test/**/*_test.js",
-    "test-debug": "cross-env NODE_ENV=test mocha --require @babel/register --require ./test/setup-jsdom.js --debug-brk --inspect test/Form_test.js"
+    "test-coverage": "cross-env NODE_ENV=test nyc --reporter=lcov mocha --require @babel/register --require ./test/setup-jsdom.js test/**/*_test.js"
   },
   "lint-staged": {
     "{src,test}/**/*.js": [

packages/core/src/utils.js

@@ -1038,7 +1038,10 @@ export function toPathSchema(schema, name = "", rootSchema, formData = {}) {
     return toPathSchema(_schema, name, rootSchema, formData);
   }
 
-  if (schema.hasOwnProperty("additionalProperties")) {
+  if (
+    schema.hasOwnProperty("additionalProperties") &&
+    schema.additionalProperties !== false
+  ) {
     pathSchema.__rjsf_additionalProperties = true;
   }
 

packages/core/test/Form_test.js

@@ -3193,6 +3193,45 @@ describe("Form omitExtraData and liveOmit", () => {
     });
   });
 
+  it("should remove extra data on change with omitExtraData=true and liveOmit=true if additionalProperties equal false", () => {
+    const omitExtraData = true;
+    const liveOmit = true;
+    const schema = {
+      type: "object",
+      properties: {
+        foo: { type: "string", additionalProperties: false },
+        bar: { type: "string", additionalProperties: false },
+        info: {
+          type: "object",
+          additionalProperties: false,
+          properties: {
+            name: {
+              type: "string",
+              additionalProperties: false,
+            },
+          },
+          required: ["name"],
+        },
+      },
+      additionalProperties: false,
+    };
+    const formData = {
+      foo: "foo",
+      baz: "baz",
+      info: { majorVersion: "v1123", name: "foofoo" },
+    };
+    const { node } = createFormComponent({
+      schema,
+      formData,
+      omitExtraData,
+      liveOmit,
+    });
+
+    Simulate.submit(node);
+
+    expect(node.querySelectorAll(".errors li")).to.have.length.of(0);
+  });
+
   it("should rename formData key if key input is renamed in a nested object with omitExtraData=true and liveOmit=true", () => {
     const { node, onChange } = createFormComponent(
       {