From d7f750577711810d40f1720e26c47be86b8bf5f0 Mon Sep 17 00:00:00 2001
From: Sun <1652829889@qq.com>
Date: Thu, 18 Apr 2024 13:50:12 +0800
Subject: [PATCH] fix: duplicate triggering of the click event (#29)

* fix(reset): duplicate triggering of the click event

* fix(submit): duplicate triggering of the click event
---
 packages/components/src/reset/index.tsx  |  7 ++++---
 packages/components/src/submit/index.tsx | 11 +++++++----
 2 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/packages/components/src/reset/index.tsx b/packages/components/src/reset/index.tsx
index 28eb1aa..cd8be6e 100644
--- a/packages/components/src/reset/index.tsx
+++ b/packages/components/src/reset/index.tsx
@@ -1,7 +1,7 @@
 import { observer } from '@formily/reactive-vue'
 import { useParentForm } from '@formily/vue'
 import { Button } from 'ant-design-vue'
-import { defineComponent } from 'vue'
+import { defineComponent, type PropType } from 'vue'
 
 export const Reset = observer(
   defineComponent({
@@ -14,7 +14,8 @@ export const Reset = observer(
       validate: {
         type: Boolean,
         default: false
-      }
+      },
+      onClick: Function as PropType<(event: MouseEvent) => void | boolean>
     },
     emits: ['resetValidateSuccess', 'resetValidateFailed'],
     setup(props, { attrs, slots, emit }) {
@@ -25,7 +26,7 @@ export const Reset = observer(
           <Button
             {...attrs}
             onClick={async (e) => {
-              const result = await (attrs as any).onClick?.(e)
+              const result = await props.onClick?.(e)
               if (result === false) return
               form
                 ?.reset('*', {
diff --git a/packages/components/src/submit/index.tsx b/packages/components/src/submit/index.tsx
index 782bcf2..d2e4559 100644
--- a/packages/components/src/submit/index.tsx
+++ b/packages/components/src/submit/index.tsx
@@ -2,12 +2,12 @@ import { observer } from '@formily/reactive-vue'
 import { useParentForm } from '@formily/vue'
 import type { ButtonProps } from 'ant-design-vue'
 import { Button } from 'ant-design-vue'
-import { defineComponent } from 'vue'
+import { defineComponent, type PropType } from 'vue'
 
 import type { IFormFeedback } from '@formily/core'
 
 export interface ISubmitProps extends ButtonProps {
-  onClick?: (e: MouseEvent) => any
+  onClick?: (event: MouseEvent) => void | boolean
   onSubmit?: (values: any) => any
   onSubmitSuccess?: (payload: any) => void
   onSubmitFailed?: (feedbacks: IFormFeedback[]) => void
@@ -16,11 +16,14 @@ export interface ISubmitProps extends ButtonProps {
 export const Submit = observer(
   defineComponent({
     name: 'FSubmit',
+    props: {
+      onClick: Function as PropType<ISubmitProps['onClick']>
+    },
     setup(props, { attrs, slots }) {
       const formRef = useParentForm()
 
       return () => {
-        const { onClick, onSubmit, onSubmitSuccess, onSubmitFailed } = attrs
+        const { onSubmit, onSubmitSuccess, onSubmitFailed } = attrs
         const form = formRef?.value
         return (
           <Button
@@ -28,7 +31,7 @@ export const Submit = observer(
             {...attrs}
             loading={attrs.loading !== undefined ? attrs.loading : form?.submitting}
             onClick={async (e) => {
-              const result = await (onClick as Function)?.(e)
+              const result = await props.onClick?.(e)
               if (result === false) return
               if (onSubmit) {
                 form