react-dropzone 3.13.0 → 3.13.4

package/src/getDataTransferItems.js

@@ -1,22 +1,17 @@
-export default function getDataTransferFiles(event, isMultipleAllowed = true) {
-  let dataTransferItemsList = [];
+export default function getDataTransferFiles(event) {
+  let dataTransferItemsList = []
   if (event.dataTransfer) {
-    const dt = event.dataTransfer;
+    const dt = event.dataTransfer
     if (dt.files && dt.files.length) {
-      dataTransferItemsList = dt.files;
+      dataTransferItemsList = dt.files
     } else if (dt.items && dt.items.length) {
       // During the drag even the dataTransfer.files is null
       // but Chrome implements some drag store, which is accesible via dataTransfer.items
-      dataTransferItemsList = dt.items;
+      dataTransferItemsList = dt.items
     }
   } else if (event.target && event.target.files) {
-    dataTransferItemsList = event.target.files;
+    dataTransferItemsList = event.target.files
   }
-
-  if (dataTransferItemsList.length > 0) {
-    dataTransferItemsList = isMultipleAllowed ? dataTransferItemsList : [dataTransferItemsList[0]];
-  }
-
   // Convert from DataTransferItemsList to the native Array
-  return Array.prototype.slice.call(dataTransferItemsList);
+  return Array.prototype.slice.call(dataTransferItemsList)
 }

package/src/getDataTransferItems.spec.js

@@ -1,26 +1,29 @@
-import getDataTransferFiles from './getDataTransferItems';
+import getDataTransferFiles from './getDataTransferItems'
 
-const files = [{
-  name: 'file1.pdf',
-  size: 1111,
-  type: 'application/pdf'
-}, {
-  name: 'cats.gif',
-  size: 1234,
-  type: 'image/gif'
-}, {
-  name: 'dogs.jpg',
-  size: 2345,
-  type: 'image/jpeg'
-}];
+const files = [
+  {
+    name: 'file1.pdf',
+    size: 1111,
+    type: 'application/pdf'
+  },
+  {
+    name: 'cats.gif',
+    size: 1234,
+    type: 'image/gif'
+  },
+  {
+    name: 'dogs.jpg',
+    size: 2345,
+    type: 'image/jpeg'
+  }
+]
 
 describe('getDataTransferFiles', () => {
-
   it('should return an array', () => {
-    const res = getDataTransferFiles({});
-    expect(res).toBeInstanceOf(Array);
-    expect(res).toHaveLength(0);
-  });
+    const res = getDataTransferFiles({})
+    expect(res).toBeInstanceOf(Array)
+    expect(res).toHaveLength(0)
+  })
 
   it('should get dataTransfer before using target', () => {
     const event = {
@@ -30,11 +33,11 @@ describe('getDataTransferFiles', () => {
       dataTransfer: {
         files
       }
-    };
-    const res = getDataTransferFiles(event);
-    expect(res).toBeInstanceOf(Array);
-    expect(res).toHaveLength(3);
-  });
+    }
+    const res = getDataTransferFiles(event)
+    expect(res).toBeInstanceOf(Array)
+    expect(res).toHaveLength(3)
+  })
 
   it('should use dataTransfer.items if files is not defined', () => {
     const event = {
@@ -44,22 +47,22 @@ describe('getDataTransferFiles', () => {
       dataTransfer: {
         items: files
       }
-    };
-    const res = getDataTransferFiles(event);
-    expect(res).toBeInstanceOf(Array);
-    expect(res).toHaveLength(3);
-  });
+    }
+    const res = getDataTransferFiles(event)
+    expect(res).toBeInstanceOf(Array)
+    expect(res).toHaveLength(3)
+  })
 
   it('should use event.target if dataTransfer is not defined', () => {
     const event = {
       target: {
         files
       }
-    };
-    const res = getDataTransferFiles(event);
-    expect(res).toBeInstanceOf(Array);
-    expect(res).toHaveLength(3);
-  });
+    }
+    const res = getDataTransferFiles(event)
+    expect(res).toBeInstanceOf(Array)
+    expect(res).toHaveLength(3)
+  })
 
   it('should prioritize dataTransfer.files over .files', () => {
     const event = {
@@ -67,36 +70,20 @@ describe('getDataTransferFiles', () => {
         files: [{}, {}],
         items: [{}, {}, {}]
       }
-    };
-    const res = getDataTransferFiles(event);
-    expect(res).toBeInstanceOf(Array);
-    expect(res).toHaveLength(2);
-  });
-
-  it('should return first file if isMultipleAllowed is false', () => {
-    const event = {
-      target: {
-        files
-      },
-      dataTransfer: {
-        files
-      }
-    };
-    const res = getDataTransferFiles(event, false);
-    expect(res).toHaveLength(1);
-    expect(res[0].name).toEqual(files[0].name);
-  });
+    }
+    const res = getDataTransferFiles(event)
+    expect(res).toBeInstanceOf(Array)
+    expect(res).toHaveLength(2)
+  })
 
   it('should not mutate data', () => {
     const event = {
       dataTransfer: {
         files
       }
-    };
-    expect(Object.keys(files[2])).toHaveLength(3);
-    getDataTransferFiles(event, true);
-    expect(Object.keys(files[2])).toHaveLength(3);
-
-  });
-
-});
+    }
+    expect(Object.keys(files[2])).toHaveLength(3)
+    getDataTransferFiles(event, true)
+    expect(Object.keys(files[2])).toHaveLength(3)
+  })
+})