refactoring into proper methods

3 files changed, 34 insertions, 57 deletions

diff --git a/apps/files/js/fileactions.js b/apps/files/js/fileactions.js
index 6678c2d5b34..78f195be91f 100644
--- a/apps/files/js/fileactions.js
+++ b/apps/files/js/fileactions.js
@@ -486,36 +486,10 @@
 					return;
 				}
 
-				downloadFileaction.addClass('disabled');
-				var icon = downloadFileaction.find('img');
-				var sourceImage = icon.attr('src');
-				icon.attr('src', sourceImage.replace('actions/download.svg', 'loading-small.gif'));
-
-				var randomString = Math.random().toString(36).substring(2);
-
-				var isCookieSet = function(name, value) {
-					var cookies = document.cookie.split(';');
-					for (var i=0; i < cookies.length; i++) {
-						var cookie = cookies[i].split('=');
-						if (cookie[0].trim() === name && cookie[1].trim() === value) {
-							return true;
-						}
-					}
-					return false;
-				};
-
-				var checkForDownloadCookie = function() {
-					if (!isCookieSet('ocDownloadStarted', randomString)){
-						setTimeout(checkForDownloadCookie, 500);
-					} else {
-						icon.attr('src', sourceImage);
-						downloadFileaction.removeClass('disabled');
-					}
-				};
+				var randomString = OCA.Files.Files.handleDownloadSpinner(downloadFileaction);
 
 				if (url) {
 					OC.redirect(url + '&downloadStartSecret=' + randomString);
-					checkForDownloadCookie();
 				}
 			}, t('files', 'Download'));
 		}
diff --git a/apps/files/js/filelist.js b/apps/files/js/filelist.js
index 86c6e225115..c018777d8a8 100644
--- a/apps/files/js/filelist.js
+++ b/apps/files/js/filelist.js
@@ -426,37 +426,9 @@
 				return;
 			}
 
-			downloadFileaction.addClass('disabled');
-			var icon = downloadFileaction.find('img');
-			var sourceImage = icon.attr('src');
-			icon.attr('src', sourceImage.replace('actions/download.svg', 'loading-small.gif'));
-
-			var randomString = Math.random().toString(36).substring(2);
-
-			var isCookieSet = function(name, value) {
-				var cookies = document.cookie.split(';');
-				for (var i=0; i < cookies.length; i++) {
-					var cookie = cookies[i].split('=');
-					if (cookie[0].trim() === name && cookie[1].trim() === value) {
-						return true;
-					}
-				}
-				return false;
-			};
-
-			var checkForDownloadCookie = function() {
-				console.log('check');
-				if (!isCookieSet('ocDownloadStarted', randomString)){
-					setTimeout(checkForDownloadCookie, 500);
-				} else {
-					console.log('boom');
-					icon.attr('src', sourceImage);
-					downloadFileaction.removeClass('disabled');
-				}
-			};
+			var randomString = OCA.Files.Files.handleDownloadSpinner(downloadFileaction);
 
 			OC.redirect(this.getDownloadUrl(files, dir) + '&downloadStartSecret=' + randomString);
-			checkForDownloadCookie();
 			return false;
 		},
 
diff --git a/apps/files/js/files.js b/apps/files/js/files.js
index 44868e78bd0..de372a71f74 100644
--- a/apps/files/js/files.js
+++ b/apps/files/js/files.js
@@ -276,8 +276,39 @@
 				FileList.scrollTo(getURLParameter('scrollto'));
 			}
 			*/
+		},
+
+		/**
+		 * Replaces the download icon with a loading spinner and returns token for the download check:
+		 * - browser sends download request and adds parameter with a token
+		 * - server notices this token and adds a set cookie to the download response
+		 * - browser now adds this cookie for the domain
+		 * - JS periodically checks for this cookie and then knows when the download has started to remove all the user feedback
+		 *
+		 * @param downloadButtonElement download fileaction
+		 * @returns {string} random token that needs to be set as cookie
+		 */
+		handleDownloadSpinner: function(downloadButtonElement) {
+			var randomString = Math.random().toString(36).substring(2),
+				icon = downloadButtonElement.find('img'),
+				sourceImage = icon.attr('src'),
+				checkForDownloadCookie = function() {
+					if (!OC.Util.isCookieSetToValue('ocDownloadStarted', randomString)){
+						return false;
+					} else {
+						icon.attr('src', sourceImage);
+						downloadButtonElement.removeClass('disabled');
+						return true;
+					}
+				};
+
+			downloadButtonElement.addClass('disabled');
+			icon.attr('src', sourceImage.replace('actions/download.svg', 'loading-small.gif'));
+			OC.Util.waitFor(checkForDownloadCookie, 500);
+
+			return randomString;
 		}
-	}
+	};
 
 	Files._updateStorageStatisticsDebounced = _.debounce(Files._updateStorageStatistics, 250);
 	OCA.Files.Files = Files;