fix: event return parsed file (#307)

Author: zombieJ

src/AjaxUploader.tsx

@@ -19,7 +19,7 @@ interface ParsedFileInfo {
   origin: RcFile;
   action: string;
   data: object;
-  parsedFile: Exclude<BeforeUploadFileType, boolean>;
+  parsedFile: RcFile;
 }
 
 class AjaxUploader extends Component<UploadProps> {
@@ -162,23 +162,28 @@ class AjaxUploader extends Component<UploadProps> {
       mergedData = data;
     }
 
-    const parsedFile =
+    const parsedData =
       // string type is from legacy `transformFile`.
       // Not sure if this will work since no related test case works with it
       (typeof transformedFile === 'object' || typeof transformedFile === 'string') &&
       transformedFile
         ? transformedFile
         : file;
 
-    // Used for `request.ts` get form data name
-    if (typeof parsedFile === 'object' && !(parsedFile as any).name) {
-      (parsedFile as any).name = file.name;
+    let parsedFile: File;
+    if (parsedData instanceof File) {
+      parsedFile = parsedData;
+    } else {
+      parsedFile = new File([parsedData], file.name, { type: file.type });
     }
 
+    const mergedParsedFile: RcFile = parsedFile as RcFile;
+    mergedParsedFile.uid = file.uid;
+
     return {
       origin: file,
       data: mergedData,
-      parsedFile,
+      parsedFile: mergedParsedFile,
       action: mergedAction,
     };
   };
@@ -203,17 +208,17 @@ class AjaxUploader extends Component<UploadProps> {
       method: method || 'post',
       onProgress: (e: UploadProgressEvent) => {
         const { onProgress } = this.props;
-        onProgress?.(e, origin);
+        onProgress?.(e, parsedFile);
       },
       onSuccess: (ret: any, xhr: XMLHttpRequest) => {
         const { onSuccess } = this.props;
-        onSuccess?.(ret, origin, xhr);
+        onSuccess?.(ret, parsedFile, xhr);
 
         delete this.reqs[uid];
       },
       onError: (err: UploadRequestError, ret: any) => {
         const { onError } = this.props;
-        onError?.(err, ret, origin);
+        onError?.(err, ret, parsedFile);
 
         delete this.reqs[uid];
       },

tests/uploader.spec.js

@@ -463,20 +463,7 @@ describe('uploader', () => {
   });
 
   describe('onBatchStart', () => {
-    const files = [
-      {
-        name: 'bamboo.png',
-        toString() {
-          return this.name;
-        },
-      },
-      {
-        name: 'light.png',
-        toString() {
-          return this.name;
-        },
-      },
-    ];
+    const files = [new File([], 'bamboo.png'), new File([], 'light.png')];
 
     const batchEventFiles = files.map(file =>
       expect.objectContaining({