Don't allow user to delete, rename and re-share the "Shared" directory

This commit is contained in:
Bjoern Schiessle 2012-06-04 10:42:09 +02:00
parent 8983c6dd6b
commit c8f670dfab
3 changed files with 6 additions and 4 deletions

View file

@ -51,7 +51,7 @@ FileActions={
var actions=this.get(mime,type);
return actions[name];
},
display:function(parent){
display:function(parent, filename, type){
FileActions.currentFile=parent;
$('#fileList span.fileactions, #fileList td.date a.action').remove();
var actions=FileActions.get(FileActions.getCurrentMimeType(),FileActions.getCurrentType());
@ -62,6 +62,8 @@ FileActions={
parent.children('a.name').append('<span class="fileactions" />');
var defaultAction=FileActions.getDefault(FileActions.getCurrentMimeType(),FileActions.getCurrentType());
for(name in actions){
// no rename and share action for the 'Shared' dir
if((name=='Rename' || name =='Share') && type=='dir' && filename=='Shared') { continue; }
if((name=='Download' || actions[name]!=defaultAction) && name!='Delete'){
var img=FileActions.icons[name];
if(img.call){
@ -84,7 +86,7 @@ FileActions={
parent.find('a.name>span.fileactions').append(element);
}
}
if(actions['Delete']){
if(actions['Delete'] && (type!='dir' || filename != 'Shared')){ // no delete action for the 'Shared' dir
var img=FileActions.icons['Delete'];
if(img.call){
img=img(file);

View file

@ -56,7 +56,7 @@ $(document).ready(function() {
// Sets the file-action buttons behaviour :
$('tr').live('mouseenter',function(event) {
FileActions.display($(this).children('td.filename'));
FileActions.display($(this).children('td.filename'), $(this).attr('data-file'), $(this).attr('data-type'));
});
$('tr').live('mouseleave',function(event) {
FileActions.hide();

View file

@ -224,7 +224,7 @@ class OC_Files {
* @param file $name
*/
public static function delete($dir,$file){
if(OC_User::isLoggedIn()){
if(OC_User::isLoggedIn() && ($dir!= '' || $file != 'Shared')) {
$file=$dir.'/'.$file;
return OC_Filesystem::unlink($file);
}