fix(Form)!: include nested state in submit data

nuxt · Jan 6, 2025 · 517acae · 517acae
1 parent 561fcd0
commit 517acae
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/runtime/components/Form.vue b/src/runtime/components/Form.vue
@@ -157,6 +157,8 @@ async function _validate(opts: { name?: string | string[], silent?: boolean, nes
     throw new FormValidationException(formId, errors.value, childErrors)
   }
 
+  Object.assign(props.state, parsedValue.value)
+
   return props.state as T
 }
 
@@ -169,8 +171,7 @@ async function onSubmitWrapper(payload: Event) {
   const event = payload as FormSubmitEvent<any>
 
   try {
-    await _validate({ nested: true })
-    event.data = props.schema ? parsedValue.value : props.state
+    event.data = await _validate({ nested: true })
     await props.onSubmit?.(event)
   } catch (error) {
     if (!(error instanceof FormValidationException)) {

diff --git a/test/components/Form.spec.ts b/test/components/Form.spec.ts
@@ -347,6 +347,15 @@ describe('Form', () => {
       expect(nestedField.text()).toBe('Required')
     })
 
+    test('submit event contains nested attributes', async () => {
+      state.email = 'bob@dylan.com'
+      state.password = 'strongpassword'
+      state.nested.field = 'nested'
+
+      await form.value.submit()
+      expect(wrapper.setupState.onSubmit).toHaveBeenCalledWith(expect.objectContaining({ data: { email: 'bob@dylan.com', password: 'strongpassword', nested: { field: 'nested' } } }))
+    })
+
     test('submit works when child is disabled', async () => {
       await form.value.submit()
       expect(wrapper.setupState.onError).toHaveBeenCalledTimes(1)