@homebound/form-state 2.26.5 → 2.26.7

package/dist/fields/listField.js

@@ -173,7 +173,7 @@ function newListFieldState(parentCopy, parentInstance, parentState, key, rules,
             // passing us cloned rows from the parentCopy, but `getOrCreateChildState` will
             // use the `copyMap` to recover the non-cloned rows, to avoid promoting the clone
             // into a real row.
-            parentInstance[key] = values.map((child) => getOrCreateChildState(child, opts).value);
+            parentInstance[key] = (values !== null && values !== void 0 ? values : []).map((child) => getOrCreateChildState(child, opts).value);
             // Make sure to tick first so that `setOriginalValue` sees the latest `rows`
             _tick.value++;
             // Reset originalCopy so that our dirty checks have the right # of rows.

package/dist/formState.test.js

@@ -152,6 +152,19 @@ describe("formState", () => {
         expect(state.books.rows[2].value === b1).toEqual(true);
         expect(ticks).toEqual(3);
     });
+    it("list value can refresh on undefined", () => {
+        // Given a list field that is currently []
+        const a1 = { firstName: "a1", books: [] };
+        const state = createAuthorInputState(a1);
+        // When we refresh it with undefined
+        state.set({ books: undefined }, { refreshing: true });
+        // Then it doesn't blow up, and stays []
+        expect(state.books.value).toEqual([]);
+        expect(state.value).toEqual({ firstName: "a1", books: [] });
+        // And it's not considered changed
+        expect(state.books.dirty).toBe(false);
+        expect(state.books.originalValue).toEqual([]);
+    });
     it("list value can observe changes", () => {
         const b1 = { title: "t1" };
         const a1 = { firstName: "a1", books: [b1] };

package/dist/useFormState.test.js

@@ -7,6 +7,7 @@ const mobx_1 = require("mobx");
 const mobx_react_1 = require("mobx-react");
 const react_2 = require("react");
 const FormStateApp_1 = require("./FormStateApp");
+const objectField_1 = require("./fields/objectField");
 const rules_1 = require("./rules");
 const useFormState_1 = require("./useFormState");
 describe("useFormState", () => {
@@ -109,6 +110,25 @@ describe("useFormState", () => {
         (0, rtl_utils_1.click)(r.change);
         expect(r.firstName).toHaveTextContent("0");
     });
+    it("can init with fragments", async () => {
+        // Given a component
+        function TestComponent() {
+            var _a, _b, _c, _d;
+            // And we have query data that may or may not be defined
+            const form = (0, useFormState_1.useFormState)({
+                config: authorWithBookFragments,
+                init: {
+                    input: {},
+                    map: () => ({ firstName: "a1", books: [{ id: "b:1", data: (0, objectField_1.fragment)({ misc: "data" }) }] }),
+                },
+            });
+            return ((0, jsx_runtime_1.jsxs)(jsx_runtime_1.Fragment, { children: [(0, jsx_runtime_1.jsx)("div", { "data-testid": "fieldValue", children: form.books.rows[0].data.value.misc }), (0, jsx_runtime_1.jsx)("div", { "data-testid": "rowValue", children: (_b = (_a = form.books.rows[0].value.data) === null || _a === void 0 ? void 0 : _a.misc) !== null && _b !== void 0 ? _b : "unset" }), (0, jsx_runtime_1.jsx)("div", { "data-testid": "listValue", children: (_d = (_c = form.books.value[0].data) === null || _c === void 0 ? void 0 : _c.misc) !== null && _d !== void 0 ? _d : "unset" })] }));
+        }
+        const r = await (0, rtl_utils_1.render)((0, jsx_runtime_1.jsx)(TestComponent, {}));
+        expect(r.fieldValue).toHaveTextContent("data");
+        expect(r.rowValue).toHaveTextContent("unset");
+        expect(r.listValue).toHaveTextContent("unset");
+    });
     it("keeps local changed values when a query refreshes", async () => {
         // Given a component
         function TestComponent() {
@@ -567,7 +587,7 @@ describe("useFormState", () => {
             });
             return (0, jsx_runtime_1.jsx)("button", { "data-testid": "set", onClick: () => form.firstName.set("f2") });
         }
-        const r = await (0, rtl_utils_1.render)((0, jsx_runtime_1.jsx)(TestComponent, {}));
+        await (0, rtl_utils_1.render)((0, jsx_runtime_1.jsx)(TestComponent, {}));
         // When we change a field, and try to commit, it blows up
         // await expect(clickAndWait(r.set)).rejects.toThrow(
         //   "When using autoSave, you should not manually call commitChanges",
@@ -620,6 +640,17 @@ const authorWithAddressAndBooksConfig = {
         },
     },
 };
+const authorWithBookFragments = {
+    id: { type: "value" },
+    firstName: { type: "value" },
+    books: {
+        type: "list",
+        config: {
+            id: { type: "value" },
+            data: { type: "fragment" },
+        },
+    },
+};
 /** Emulates a user focusing and then blurring a field. */
 function focusAndBlur(state) {
     state.focus();

package/dist/utils.js

@@ -85,6 +85,9 @@ function pickFields(formConfig, instance) {
         else if (keyConfig.type === "value") {
             return [key, value];
         }
+        else if (keyConfig.type === "fragment") {
+            return [key, value];
+        }
         else {
             return assertNever(keyConfig);
         }

package/package.json

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