@homebound/form-state 2.12.0 → 2.13.0

package/dist/FormStateApp.js

@@ -52,8 +52,8 @@ function FormStateApp() {
                                                     jsx_runtime_1.jsx("td", { children: formState.valid.toString() }, void 0),
                                                     jsx_runtime_1.jsx("td", { children: formState.dirty.toString() }, void 0)] }, void 0) }, void 0)] }), void 0),
                                 jsx_runtime_1.jsxs("div", { children: [jsx_runtime_1.jsx("button", Object.assign({ "data-testid": "touch", onClick: () => (formState.touched = !formState.touched) }, { children: "touch" }), void 0),
-                                        jsx_runtime_1.jsx("button", Object.assign({ "data-testid": "reset", onClick: () => formState.reset() }, { children: "reset" }), void 0),
-                                        jsx_runtime_1.jsx("button", Object.assign({ "data-testid": "save", onClick: () => formState.save() }, { children: "save" }), void 0),
+                                        jsx_runtime_1.jsx("button", Object.assign({ "data-testid": "revertChanges", onClick: () => formState.revertChanges() }, { children: "revert" }), void 0),
+                                        jsx_runtime_1.jsx("button", Object.assign({ "data-testid": "commitChanges", onClick: () => formState.commitChanges() }, { children: "commit" }), void 0),
                                         jsx_runtime_1.jsx("button", Object.assign({ "data-testid": "set", onClick: () => formState.set({ firstName: "a2" }) }, { children: "set" }), void 0)] }, void 0)] }, void 0)] }), void 0) }), void 0));
         } }, void 0));
 }

package/dist/FormStateApp.test.js

@@ -12,7 +12,7 @@ describe("FormStateApp", () => {
         expect(r.firstName_dirty()).toHaveTextContent("true");
         react_1.fireEvent.blur(r.firstName());
         expect(r.firstName_touched()).toHaveTextContent("true");
-        rtl_utils_1.click(r.save);
+        rtl_utils_1.click(r.commitChanges);
         expect(r.firstName_dirty()).toHaveTextContent("false");
         expect(r.firstName_touched()).toHaveTextContent("false");
     });
@@ -20,7 +20,7 @@ describe("FormStateApp", () => {
         const r = await rtl_utils_1.render(jsx_runtime_1.jsx(FormStateApp_1.FormStateApp, {}, void 0));
         expect(r.firstName_original()).toHaveTextContent("a1");
         rtl_utils_1.click(r.set);
-        rtl_utils_1.click(r.save);
+        rtl_utils_1.click(r.commitChanges);
         expect(r.firstName_original()).toHaveTextContent("a2");
     });
 });

package/dist/fields/listField.js

@@ -184,16 +184,14 @@ function newListFieldState(parentInstance, parentState, key, rules, listConfig,
             _tick.value++;
             maybeAutoSave();
         },
-        reset() {
+        revertChanges() {
             if (originalCopy) {
                 this.set(originalCopy, { resetting: true });
-                this.rows.forEach((r) => r.reset());
+                this.rows.forEach((r) => r.revertChanges());
             }
         },
-        save() {
-            this.rows.forEach((r) => {
-                r.save();
-            });
+        commitChanges() {
+            this.rows.forEach((r) => r.commitChanges());
             originalCopy = [...(parentInstance[key] || [])];
             _tick.value++;
         },

package/dist/fields/objectField.js

@@ -132,12 +132,12 @@ function newObjectState(config, parentState, parentListState, instance, key, may
             });
         },
         // Resets all fields back to their original values
-        reset() {
-            getFields(this).forEach((f) => f.reset());
+        revertChanges() {
+            getFields(this).forEach((f) => f.revertChanges());
         },
         // Saves all current values into _originalValue
-        save() {
-            getFields(this).forEach((f) => f.save());
+        commitChanges() {
+            getFields(this).forEach((f) => f.commitChanges());
         },
         // Create a result that is only populated with changed keys
         get changedValue() {

package/dist/fields/valueField.d.ts

@@ -33,9 +33,9 @@ export interface FieldState<T, V> {
     maybeAutoSave(): void;
     set(value: V, opts?: SetOpts): void;
     /** Reverts back to the original value and resets dirty/touched. */
-    reset(): void;
+    revertChanges(): void;
     /** Accepts the current changed value (if any) as the original and resets dirty/touched. */
-    save(): void;
+    commitChanges(): void;
 }
 /** Public options for our `set` command. */
 export interface SetOpts {

package/dist/fields/valueField.js

@@ -118,13 +118,13 @@ function newValueFieldState(parentInstance, parentState, key, rules, isIdKey, is
                 this.maybeAutoSave();
             }
         },
-        reset() {
+        revertChanges() {
             if (!computed) {
                 this.set(this.originalValue, { resetting: true });
             }
             this.touched = false;
         },
-        save() {
+        commitChanges() {
             if (is_plain_object_1.isPlainObject(this.originalValue)) {
                 this.originalValue = mobx_1.toJS(this.value);
             }

package/dist/formState.test.js

@@ -452,7 +452,7 @@ describe("formState", () => {
         a1.set({ firstName: "first" });
         expect(maybeAutoSave).toBeCalledTimes(1);
         // When we reset
-        a1.reset();
+        a1.revertChanges();
         // We don't call blur again
         expect(maybeAutoSave).toBeCalledTimes(1);
     });
@@ -541,7 +541,7 @@ describe("formState", () => {
         a1.books.add({ title: "b3" });
         expect(a1.books.touched).toEqual(true);
         expect(a1.dirty).toBeTruthy();
-        a1.reset();
+        a1.revertChanges();
         expect(a1.firstName.value).toBe("a1");
         expect(a1.firstName.touched).toBeFalsy();
         expect(a1.lastName.value).toBe("aL1");
@@ -577,7 +577,7 @@ describe("formState", () => {
         // And the "readOnly=true" operation has "beat" the reset operation
         a1.readOnly = true;
         // When we reset
-        a1.reset();
+        a1.revertChanges();
         // Then it still works
         expect(a1.firstName.value).toBe("a1");
         expect(a1.firstName.touched).toBeFalsy();
@@ -607,15 +607,15 @@ describe("formState", () => {
         a1.books.rows[1].set({ title: "bb3" });
         // verify ValueFieldState is dirty, then save, then no longer dirty.
         expect(a1.firstName.dirty).toBeTruthy();
-        a1.firstName.save();
+        a1.firstName.commitChanges();
         expect(a1.firstName.dirty).toBeFalsy();
         // verify ListFieldState is dirty, then save, then no longer dirty.
         expect(a1.books.dirty).toBeTruthy();
-        a1.books.save();
+        a1.books.commitChanges();
         expect(a1.books.dirty).toBeFalsy();
         // Verify the remaining form is still dirty
         expect(a1.dirty).toBeTruthy();
-        a1.save();
+        a1.commitChanges();
         // Verify after save the whole form is no longer dirty.
         expect(a1.dirty).toBeFalsy();
     });
@@ -839,7 +839,7 @@ describe("formState", () => {
         expect(a.books.rows[1].readOnly).toBeFalsy();
         expect(a.books.rows[1].title.readOnly).toBeFalsy();
         // And reset does not blow up
-        a.reset();
+        a.revertChanges();
     });
     it("can set nested values when original null", () => {
         const a = createAuthorWithAddressInputState({ address: null });
@@ -894,7 +894,7 @@ describe("formState", () => {
         expect(formState.fullName.value).toEqual("first last");
         formState.firstName.value = "change";
         expect(formState.fullName.value).toEqual("change last");
-        formState.reset();
+        formState.revertChanges();
         expect(formState.firstName.value).toEqual("first");
     });
     it("can set computeds that have setters", () => {
@@ -907,7 +907,7 @@ describe("formState", () => {
         expect(formState.firstName.value).toEqual("Bob");
         formState.fullName.set("Fred Smith");
         expect(formState.firstName.value).toEqual("Fred");
-        formState.reset();
+        formState.revertChanges();
         expect(formState.firstName.value).toEqual("first");
     });
     it("can mark fields as read only", () => {
@@ -920,7 +920,7 @@ describe("formState", () => {
         expect(formState.fullName.readOnly).toEqual(true);
         formState.firstName.value = "change";
         expect(formState.fullName.value).toEqual("change last");
-        formState.reset();
+        formState.revertChanges();
         expect(formState.firstName.value).toEqual("first");
     });
     it("lets rules validate against other fields", () => {
@@ -1300,7 +1300,7 @@ describe("formState", () => {
         // Then the list state is dirty
         expect(a1.books.dirty).toBeTruthy();
         // When saving the form state
-        a1.save();
+        a1.commitChanges();
         // Then the list is no longer dirty
         expect(a1.books.dirty).toBeFalsy();
         // When adding a new book

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@homebound/form-state",
-  "version": "2.12.0",
+  "version": "2.13.0",
   "author": "Homebound",
   "license": "MIT",
   "main": "dist/index.js",