Fixed tests

Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>

3 files changed, 8 insertions, 72 deletions

diff --git a/apps/files/js/app.js b/apps/files/js/app.js
index d407c49ea51..52c92645b2d 100644
--- a/apps/files/js/app.js
+++ b/apps/files/js/app.js
@@ -239,7 +239,7 @@
 			var params;
 			if (e && e.itemId) {
 				params = {
-					view: e.view !== '' ? e.view : e.itemId,
+					view: typeof e.view === 'string' && e.view !== '' ? e.view : e.itemId,
 					dir: e.dir ? e.dir : '/'
 				};
 				this._changeUrl(params.view, params.dir);
diff --git a/apps/files/js/navigation.js b/apps/files/js/navigation.js
index d783fcc2156..d4fa06cb45e 100644
--- a/apps/files/js/navigation.js
+++ b/apps/files/js/navigation.js
@@ -81,8 +81,6 @@
 		_setupEvents: function () {
 			this.$el.on('click', 'li a', _.bind(this._onClickItem, this))
 			this.$el.on('click', 'li button', _.bind(this._onClickMenuButton, this));
-			this._setOnDrag();
-
 		},
 
 		/**
@@ -135,7 +133,7 @@
 			}
 			this._activeItem = itemId;
 			currentItem.children('a').addClass('active');
-			this.$currentContent = $('#app-content-' + (itemView !== '' ? itemView : itemId));
+			this.$currentContent = $('#app-content-' + (typeof itemView === 'string' && itemView !== '' ? itemView : itemId));
 			this.$currentContent.removeClass('hidden');
 			if (!options || !options.silent) {
 				this.$currentContent.trigger(jQuery.Event('show'));
@@ -170,74 +168,6 @@
 		},
 
 		/**
-		 * Event handler for when dragging an item
-		 */
-		_setOnDrag: function () {
-			var scope = this;
-			var element = $("#sublist-favorites");
-			$(function () {
-				if (document.getElementById(scope.$quickAccessListKey.toString()).hasAttribute("draggable")) {
-					element.sortable({
-						axis: "y",
-						containment: "parent",
-						scroll: false,
-						zIndex: 0,
-						opacity: 0.5,
-						tolerance: "pointer",
-						//revert: 0.05,
-						//delay: 150,
-						start: function (event, ui) {
-							//Fix for offset
-							ui.helper[0].style.left = '0px';
-
-							//Change Icon while dragging
-							var list = document.getElementById(scope.$quickAccessListKey).getElementsByTagName('li');
-							for (var j = 0; j < list.length; j++) {
-								if (!(typeof list[j].getElementsByTagName('a')[0] === 'undefined')) {
-									list[j].getElementsByTagName('a')[0].classList.remove("nav-icon-files");
-									list[j].getElementsByTagName('a')[0].classList.add('icon-menu');
-								}
-							}
-						},
-						stop: function (event, ui) {
-							//Clean up offset
-							ui.item.removeAttr("style");
-
-							//Change Icon back after dragging
-							var list = document.getElementById(scope.$quickAccessListKey.toString()).getElementsByTagName('li');
-							for (var j = 0; j < list.length; j++) {
-								if (!(typeof list[j].getElementsByTagName('a')[0] === 'undefined')) {
-									list[j].getElementsByTagName('a')[0].classList.add("nav-icon-files");
-									list[j].getElementsByTagName('a')[0].classList.remove('icon-menu');
-								}
-							}
-						},
-						update: function (event, ui) {
-							var list = document.getElementById(scope.$quickAccessListKey.toString()).getElementsByTagName('li');
-							var string = [];
-							for (var j = 0; j < list.length; j++) {
-								var Object = {
-									id: j,
-									name: scope.getCompareValue(list, j, 'alphabet')
-								};
-								string.push(Object);
-							}
-							var resultorder = JSON.stringify(string);
-							$.get(OC.generateUrl("/apps/files/api/v1/quickaccess/set/CustomSortingOrder"), {
-								order: resultorder
-							}, function (data, status) {
-							});
-						}
-					});
-				} else {
-					if (scope.$sortingStrategy === 'customorder') {
-						scope.$sortingStrategy = 'datemodified';
-					}
-				}
-			});
-		},
-
-		/**
 		 * Event handler for clicking a button
 		 */
 		_onClickMenuButton: function (ev) {
diff --git a/apps/files/js/tagsplugin.js b/apps/files/js/tagsplugin.js
index 3a1378cc9ad..bc1396b5104 100644
--- a/apps/files/js/tagsplugin.js
+++ b/apps/files/js/tagsplugin.js
@@ -72,6 +72,9 @@
 		var quickAccessList = 'sublist-favorites';
 		var collapsibleButtonId = 'button-collapse-favorites';
 		var listULElements = document.getElementById(quickAccessList);
+		if (!listULElements) {
+			return;
+		}
 		var listLIElements = listULElements.getElementsByTagName('li');
 
 		var apppath=appfolder;
@@ -101,6 +104,9 @@
 		var quickAccessList = 'sublist-favorites';
 		var collapsibleButtonId = 'button-collapse-favorites';
 		var listULElements = document.getElementById(quickAccessList);
+		if (!listULElements) {
+			return;
+		}
 		var listLIElements = listULElements.getElementsByTagName('li');
 
 		var appName = appfolder.substring(appfolder.lastIndexOf("/") + 1, appfolder.length);