Commit 86689ef4 authored by Vincent Petry's avatar Vincent Petry
Browse files

Disable JS plugin for sharing in public mode

This removes the logic that registers the share action and modifies the
rows. Share actions aren't needed in the public file list.
parent bd888748
......@@ -25,7 +25,7 @@
* @param {OCA.Files.FileList} fileList file list to be extended
*/
attach: function(fileList) {
if (fileList.id === 'trashbin') {
if (fileList.id === 'trashbin' || fileList.id === 'files.public') {
return;
}
var fileActions = fileList.fileActions;
......
......@@ -27,7 +27,7 @@ describe('OCA.Sharing.Util tests', function() {
$('#testArea').append($content);
// dummy file list
var $div = $(
'<div>' +
'<div id="listContainer">' +
'<table id="filestable">' +
'<thead></thead>' +
'<tbody id="fileList"></tbody>' +
......@@ -450,5 +450,29 @@ describe('OCA.Sharing.Util tests', function() {
.toEqual('User four, User one, User three, User two, +6');
});
});
describe('Excluded lists', function() {
function createListThenAttach(listId) {
var fileActions = new OCA.Files.FileActions();
fileList.destroy();
fileList = new OCA.Files.FileList(
$('#listContainer'), {
id: listId,
fileActions: fileActions
}
);
OCA.Sharing.Util.attach(fileList);
fileList.setFiles(testFiles);
return fileList;
}
it('does not attach to trashbin or public file lists', function() {
createListThenAttach('trashbin');
expect($('.action-share').length).toEqual(0);
expect($('[data-share-recipient]').length).toEqual(0);
createListThenAttach('files.public');
expect($('.action-share').length).toEqual(0);
expect($('[data-share-recipient]').length).toEqual(0);
});
});
});
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment