From d56e168df9eb10f047a565b7963385133f298bb3 Mon Sep 17 00:00:00 2001
From: yujiangping <yujiangping@taofen8.com>
Date: Thu, 22 Jan 2026 20:58:49 +0800
Subject: [PATCH] fix(web): improve file removal confirmation flow in
 UploadFiles

- Move custom onRemove callback execution into confirmation dialog's onOk handler
- Add async/await support for Promise-based onRemove callbacks
- Display confirmation dialog before executing removal logic to prevent accidental deletions
- Ensure file is only removed after user confirms and custom callback completes
- Improve UX by confirming user intent before triggering removal callbacks
---
 web/src/components/Upload/UploadFiles.tsx | 24 +++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)

diff --git a/web/src/components/Upload/UploadFiles.tsx b/web/src/components/Upload/UploadFiles.tsx
index a725278c..74b6a452 100644
--- a/web/src/components/Upload/UploadFiles.tsx
+++ b/web/src/components/Upload/UploadFiles.tsx
@@ -89,21 +89,25 @@ const UploadFiles = forwardRef<UploadFilesRef, UploadFilesProps>(({
 
   // 处理文件移除
   const handleRemove = (file: UploadFile) => {
-    // 如果有自定义的 onRemove 回调，先执行它
-    if (customOnRemove) {
-      const result = customOnRemove(file);
-      // 如果返回 false，阻止移除
-      if (result === false) {
-        return false;
-      }
-    }
-    
+    // 显示确认弹窗
     confirm({
       title: `${t('common.confirmRemoveFile')}`,
       okText: `${t('common.confirm')}`,
       okType: 'danger',
       cancelText: `${t('common.cancel')}`,
-      onOk: () => {
+      onOk: async () => {
+        // 如果有自定义的 onRemove 回调，在确认后执行
+        if (customOnRemove) {
+          const result = customOnRemove(file);
+          // 等待 Promise 结果
+          const finalResult = result instanceof Promise ? await result : result;
+          // 如果返回 false，阻止移除
+          if (finalResult === false) {
+            return;
+          }
+        }
+        
+        // 移除文件
         const newFileList = fileList.filter((item) => item.uid !== file.uid);
         setFileList(newFileList);
         onChange?.(newFileList);