From 0736bfb43a0ecc81ba1eb38c4b32fea8ed454957 Mon Sep 17 00:00:00 2001
From: Vincent Petry <pvince81@owncloud.com>
Date: Fri, 4 Oct 2013 17:49:42 +0200
Subject: [PATCH] Do not call changeDirectory() when no dir set on breadcrumb
Some apps like the files_trashbin app do not set a directory on its
"home" breadcrumb link.
This fix makes sure that the click event doesn't do anything in that
case and lets the browser open the link.
This fixes the "home" icon in the trashbin app which now correctly
reopens the files app.
---
apps/files/js/files.js | 9 ++++++---
1 file changed, 6 insertions(+), 3 deletions(-)
diff --git a/apps/files/js/files.js b/apps/files/js/files.js
index 899bc6469e5..09f5d6f114c 100644
--- a/apps/files/js/files.js
+++ b/apps/files/js/files.js
@@ -701,7 +701,10 @@ function checkTrashStatus() {
}
function onClickBreadcrumb(e){
- var $el = $(e.target).closest('.crumb');
- e.preventDefault();
- FileList.changeDirectory(decodeURIComponent($el.data('dir')));
+ var $el = $(e.target).closest('.crumb'),
+ $targetDir = $el.data('dir');
+ if ($targetDir !== undefined){
+ e.preventDefault();
+ FileList.changeDirectory(decodeURIComponent($targetDir));
+ }
}
--
GitLab