populate form setValues to subForm (#2801)

* fix: npm run watch on packages * populate setValues to subForm * back to deep copy --------- Co-authored-by: hoppe <hoppewang@microsoft.com>
Azure · Sep 14, 2023 · 02e6dff · 02e6dff
1 parent 40b3354
commit 02e6dff
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 0 deletions.
diff --git a/packages/bonito-core/src/form/__tests__/form.spec.ts b/packages/bonito-core/src/form/__tests__/form.spec.ts
@@ -284,6 +284,18 @@ describe("Form tests", () => {
                 },
             }
         );
+
+        // form.setValues() should set the values of the subForm
+        form.setValues({
+            ...form.values,
+            answers: {
+                color: "yellow",
+            },
+        });
+
+        expect(subForm.values).toEqual({
+            color: "yellow",
+        });
     });
 
     test("Validation", async () => {

diff --git a/packages/bonito-core/src/form/internal/form-impl.ts b/packages/bonito-core/src/form/internal/form-impl.ts
@@ -256,6 +256,11 @@ export class FormImpl<V extends FormValues> implements Form<V> {
             return;
         }
         this._values = values;
+        for (const e of this.allEntries()) {
+            if (e instanceof SubForm) {
+                e.setValues(values[e.name]);
+            }
+        }
         this._formValuesChanged(values, oldValues);
     }