holm/egroupware
1
1
Fork 0
forked from extern/egroupware
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Be more careful with checking for target path, could be user selected a directory not a file

Browse Source
This commit is contained in:
Nathan Gray 2014-11-24 19:47:48 +00:00
parent 5a2c3a530d
commit d0331ca837
1 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
filemanager/js/app.js
View File

@ -449,7 +449,16 @@ app.classes.filemanager = AppJS.extend(
if(action)
{
var paths = this._elems2paths(selected);
if(paths[0]) path = this.dirname(paths[0]);
if(paths[0]) path = paths[0];
// check if target is a file --> use it's directory instead
if(selected[0].id || path)
{
var data = egw.dataGetUIDdata(selected[0].id || 'filemanager::'+path );
if (data && data.data.mime != 'httpd/unix-directory')
{
path = this.dirname(path);
}
}
}
this._do_action('createdir', dir, true, path); // true=synchronous request
this.change_dir((path == '/' ? '' : path)+'/'+dir);