fix: preventDefault logic

Author: zombieJ

src/AjaxUploader.tsx

@@ -66,12 +66,16 @@ class AjaxUploader extends Component<UploadProps> {
     }
   };
 
-  onDataTransferFiles = async (dataTransfer: DataTransfer) => {
+  onDataTransferFiles = async (dataTransfer: DataTransfer, existFileCallback?: () => void) => {
     const { multiple, accept, directory } = this.props;
 
     const items: DataTransferItem[] = [...(dataTransfer.items || [])];
     let files: File[] = [...(dataTransfer.files || [])];
 
+    if (files.length > 0 || items.some(item => item.kind === 'file')) {
+      existFileCallback?.();
+    }
+
     if (directory) {
       files = await traverseFileTree(Array.prototype.slice.call(items), (_file: RcFile) =>
         attrAccept(_file, this.props.accept),
@@ -97,7 +101,9 @@ class AjaxUploader extends Component<UploadProps> {
 
     if (e.type === 'paste') {
       const clipboardData = (e as ClipboardEvent).clipboardData;
-      return this.onDataTransferFiles(clipboardData);
+      return this.onDataTransferFiles(clipboardData, () => {
+        e.preventDefault();
+      });
     }
   };
 

tests/uploader.spec.tsx

@@ -735,6 +735,12 @@ describe('uploader', () => {
           },
         ],
       };
+
+      const preventDefaultSpy = jest.spyOn(Event.prototype, 'preventDefault');
+
+      fireEvent.dragOver(input);
+      expect(preventDefaultSpy).toHaveBeenCalledTimes(1);
+
       fireEvent.drop(input, { dataTransfer: { items: [makeDataTransferItemAsync(files)] } });
       const mockStart = jest.fn();
       handlers.onStart = mockStart;
@@ -743,6 +749,8 @@ describe('uploader', () => {
         expect(mockStart.mock.calls.length).toBe(1);
         done();
       }, 1000);
+
+      preventDefaultSpy.mockRestore();
     });
 
     it('unaccepted type files to upload will not trigger onStart when select directory', done => {