fix: reset field should not validate closes #4323

logaretm · Jun 23, 2023 · 273cca7 · 273cca7
1 parent fc41691
commit 273cca7
Show file tree

Hide file tree

Showing 3 changed files with 27 additions and 1 deletion.
diff --git a/.changeset/soft-chicken-explain.md b/.changeset/soft-chicken-explain.md
@@ -0,0 +1,5 @@
+---
+'vee-validate': patch
+---
+
+fix: reset field should not validate closes #4323
diff --git a/packages/vee-validate/src/useFieldState.ts b/packages/vee-validate/src/useFieldState.ts
@@ -166,7 +166,7 @@ export function _useFieldValue<TValue = unknown>(
       return getFromPath<TValue>(form.values, unref(path)) as TValue;
     },
     set(newVal) {
-      form.setFieldValue(unref(path), newVal);
+      form.setFieldValue(unref(path), newVal, false);
     },
   }) as Ref<TValue>;
 

diff --git a/packages/vee-validate/tests/useField.spec.ts b/packages/vee-validate/tests/useField.spec.ts
@@ -917,4 +917,25 @@ describe('useField()', () => {
     expect(field.errors.value).toHaveLength(2);
     expect(field.errors.value).toEqual([REQUIRED_MESSAGE, 'second']);
   });
+
+  // #4323
+  test('resetField should not validate', async () => {
+    let field!: FieldContext;
+    const validator = vi.fn(val => (val ? true : REQUIRED_MESSAGE));
+
+    mountWithHoc({
+      setup() {
+        useForm();
+        field = useField('field', validator);
+
+        return {};
+      },
+    });
+
+    await flushPromises();
+    expect(field.errors.value).toHaveLength(0);
+    field.resetField({ value: '' });
+    await flushPromises();
+    expect(field.errors.value).toHaveLength(0);
+  });
 });