mbk-lab
/
nextcloud
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

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.
This commit is contained in:
Vincent Petry 2013-10-04 17:49:42 +02:00
parent fd34c969d2
commit 0736bfb43a
1 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
apps/files/js/files.js
View File

@ -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));
}
}