Cast targetPath to String when recovering it from the HTML

HTML5 data properties can be anything and if a folder only contains numbers as its name it will be considered a number and String operations will fail
owncloud · Mar 10, 2016 · 48b822b · 48b822b
1 parent 97a8c88
commit 48b822b
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/js/breadcrumb.js b/js/breadcrumb.js
@@ -86,7 +86,7 @@
 			if (!$target.is('.crumb')) {
 				$target = $target.closest('.crumb');
 			}
-			var targetPath = $(event.target).data('dir');
+			var targetPath = $(event.target).data('dir').toString();
 			var dir = Gallery.currentAlbum;
 
 			while (dir.substr(0, 1) === '/') {//remove extra leading /'s

diff --git a/js/galleryalbum.js b/js/galleryalbum.js
@@ -52,7 +52,7 @@
 			var $item = ui.draggable;
 			var $clone = ui.helper;
 			var $target = $(event.target);
-			var targetPath = $target.data('dir');
+			var targetPath = $target.data('dir').toString();
 			var filePath = $item.data('path');
 			var fileName = OC.basename(filePath);