From 69d2d0178a46b72bea2d4618bbd2f42456b9e761 Mon Sep 17 00:00:00 2001
From: Roeland Jago Douma <roeland@famdouma.nl>
Date: Thu, 24 Aug 2017 14:55:17 +0200
Subject: [PATCH] Don't try the actual file upload if the checks already error
 out

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
---
 apps/files/js/file-upload.js        | 3 +++
 apps/files/js/filelist.js           | 1 +
 apps/files/tests/js/filelistSpec.js | 2 ++
 3 files changed, 6 insertions(+)

diff --git a/apps/files/js/file-upload.js b/apps/files/js/file-upload.js
index 1f99d561a13..b86b42bdb9a 100644
--- a/apps/files/js/file-upload.js
+++ b/apps/files/js/file-upload.js
@@ -1113,6 +1113,9 @@ OC.Uploader.prototype = _.extend({
 				});
 				fileupload.on('fileuploaddrop', function(e, data) {
 					self.trigger('drop', e, data);
+					if (e.isPropagationStopped()) {
+						return false;
+					}
 				});
 
 			}
diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js
index 9bf25c5904d..79ec8e2d9f9 100644
--- a/apps/files/js/filelist.js
+++ b/apps/files/js/filelist.js
@@ -2795,6 +2795,7 @@
 					var isCreatable = (self.getDirectoryPermissions() & OC.PERMISSION_CREATE) !== 0;
 					if (!isCreatable) {
 						self._showPermissionDeniedNotification();
+						e.stopPropagation();
 						return false;
 					}
 
diff --git a/apps/files/tests/js/filelistSpec.js b/apps/files/tests/js/filelistSpec.js
index 41d202e9323..a12c0ff49b6 100644
--- a/apps/files/tests/js/filelistSpec.js
+++ b/apps/files/tests/js/filelistSpec.js
@@ -2810,6 +2810,8 @@ describe('OCA.Files.FileList tests', function() {
 						target: $target
 					},
 					preventDefault: function () {
+					},
+					stopPropagation: function() {
 					}
 				};
 				uploader.trigger('drop', eventData, data || {});