Skip to content

Commit 0b07303

Browse files
committed
Don't remove file or folder from the "shared by link" section
1 parent 4bf6666 commit 0b07303

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

apps/files_sharing/js/app.js

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ OCA.Sharing.App = {
5757
}
5858

5959
var fileActions = this._createFileActions();
60-
this._adjustOutShareActions(fileActions);
60+
this._adjustOutAndLinkShareActions(fileActions);
6161

6262
this._outFileList = new OCA.Sharing.FileList(
6363
$el,
@@ -82,13 +82,17 @@ OCA.Sharing.App = {
8282
if (this._linkFileList) {
8383
return this._linkFileList;
8484
}
85+
86+
var fileActions = this._createFileActions();
87+
this._adjustOutAndLinkShareActions(fileActions);
88+
8589
this._linkFileList = new OCA.Sharing.FileList(
8690
$el,
8791
{
8892
id: 'shares.link',
8993
scrollContainer: $('#app-content'),
9094
linksOnly: true,
91-
fileActions: this._createFileActions(),
95+
fileActions: fileActions,
9296
config: OCA.Files.App.getFilesConfig()
9397
}
9498
);
@@ -328,11 +332,11 @@ OCA.Sharing.App = {
328332
});
329333
},
330334

331-
_adjustOutShareActions: function(fileActions) {
335+
_adjustOutAndLinkShareActions: function(fileActions) {
332336
fileActions.addAdvancedFilter(function(actions, context) {
333337
// delete action is removed to prevent confusion. Users might accidentally
334338
// delete a file or a folder instead of unsharing it.
335-
// If the user clicks in the a folder, he'll be moved to the "all files" section,
339+
// If the user clicks in the folder, he'll be moved to the "all files" section,
336340
// so there is no need check for additional context and we'll remove the delete
337341
// action from all the files and folders in the list.
338342
delete(actions.Delete);

0 commit comments

Comments
 (0)