perf: 增加afterRemove判定，delRequest返回false则不执行后面的逻辑

Closes #333
fast-crud · Jan 25, 2024 · 448b2d8 · 448b2d8
1 parent 44d127f
commit 448b2d8
Show file tree

Hide file tree

Showing 4 changed files with 41 additions and 6 deletions.
diff --git a/docs/zh/api/crud-options/table.md b/docs/zh/api/crud-options/table.md
@@ -187,6 +187,26 @@
 * 说明：取消后的操作
 * 类型：`(context)=>Promise<any>`
 
+## remove.afterRemove
+* 说明：删除请求提交后的操作，返回false，不执行后面的逻辑
+* 类型：`(context:{index,row,res})=>Promise<any>`
+
+```ts
+{
+  table:{
+    remove:{
+         afterRemove:async (context)=>{
+            //context参数中带请求返回值（res），你可以在此处处理一些业务逻辑，比如校验是否失败;如果此方法返回false，不执行后面的逻辑，比如弹出删除成功消息
+            if(context.res.code !== 200){
+                return false
+            }
+        }
+    }
+  }    
+}
+```
+
+```
 
 ## remove.showSuccessNotification
 * 说明：是否显示成功删除提示

diff --git a/packages/fast-crud/src/d/crud.ts b/packages/fast-crud/src/d/crud.ts
@@ -1,6 +1,6 @@
 import { Component, Ref, ShallowRef } from "vue";
 import { ComputeContext } from "./compute";
-import { CrudExpose, DoRemoveContext } from "../d/expose";
+import { CrudExpose, DoRemoveContext, OnAfterRemoveContext } from "../d/expose";
 
 import { RuleItem } from "async-validator";
 import { UiSlot, UiSlotRet } from "@fast-crud/ui-interface";
@@ -457,6 +457,12 @@ export type RemoveProps = {
    */
   refreshTable?: boolean;
 
+  /**
+   * 删除请求之后的操作，如果返回false,终止后续的执行，比如显示删除成功通知等
+   * @param context
+   */
+  afterRemove?: (context: OnAfterRemoveContext) => Promise<any>;
+
   /**
    * 显示成功提示
    */

diff --git a/packages/fast-crud/src/d/expose.ts b/packages/fast-crud/src/d/expose.ts
@@ -270,6 +270,7 @@ export type SetSearchFormDataProps = { form: any; mergeForm?: boolean; triggerSe
  * crudExpose.doRemove参数
  */
 export type DoRemoveContext = { index?: number; row?: any };
+export type OnAfterRemoveContext = DoRemoveContext & { res: any };
 /**
  * crudExpose.doSearch参数
  */

diff --git a/packages/fast-crud/src/use/use-expose.ts b/packages/fast-crud/src/use/use-expose.ts
@@ -190,8 +190,9 @@ function useEditable(props: UseEditableProps) {
           } else {
             if (crudBinding.value.mode.name === "local") {
               editable.removeRow(editableId);
+              return { isLocal: true };
             } else {
-              await crudBinding.value.request.delRequest({ row: rowData });
+              return res;
             }
           }
         }
@@ -652,17 +653,24 @@ export function useExpose(props: UseExposeProps): UseExposeRet {
       let res = null;
       const isLocal = crudBinding.value.mode?.name === "local";
       if (opts?.handle) {
-        const ctn = await opts.handle(context);
-        if (ctn === false) {
-          return;
-        }
+        res = await opts.handle(context);
       } else {
         if (isLocal) {
           crudExpose.removeTableRow(context?.index);
         } else {
           res = await crudBinding.value.request.delRequest(context);
         }
       }
+      if (res === false) {
+        return;
+      }
+      const removeScope = { ...context, res };
+      if (removeBinding.afterRemove) {
+        const success = await removeBinding.afterRemove(removeScope);
+        if (success === false) {
+          return false;
+        }
+      }
 
       if (removeBinding.showSuccessNotification !== false) {
         ui.notification.success(t("fs.rowHandle.remove.success"));