fix: don't upload folders

AidasK · AidasK · commit baa0acd0eb9d · 2013-09-24T18:08:54.000+03:00
diff --git a/src/resumable.js b/src/resumable.js
@@ -451,7 +451,7 @@
       each(fileList, function (file) {
         // Directories have size `0` and name `.`
         // Ignore already added files
-        if (!(!file.size && (file.name === '.' || file.fileName === '.')) &&
+        if (!(file.size % 4096 === 0 && (file.name === '.' || file.fileName === '.')) &&
           !this.getFromUniqueIdentifier(this.generateUniqueIdentifier(file))) {
           var f = new ResumableFile(this, file);
           if (this.fire('fileAdded', f, event)) {
diff --git a/test/uploadSpec.js b/test/uploadSpec.js
@@ -335,4 +335,23 @@ describe('upload file', function() {
     expect(file.isUploading()).toBe(false);
     expect(file.isComplete()).toBe(true);
   });
+
+  it('should not upload folder', function () {
+    // http://stackoverflow.com/questions/8856628/detecting-folders-directories-in-javascript-filelist-objects
+    resumable.addFile({
+      name: '.',
+      size: 0
+    });
+    expect(resumable.files.length).toBe(0);
+    resumable.addFile({
+      name: '.',
+      size: 4096
+    });
+    expect(resumable.files.length).toBe(0);
+    resumable.addFile({
+      name: '.',
+      size: 4096 * 2
+    });
+    expect(resumable.files.length).toBe(0);
+  });
 });
