diff options
| author | Vincent Petry <vincent@nextcloud.com> | 2022-08-26 11:09:18 +0200 |
|---|---|---|
| committer | Vincent Petry <vincent@nextcloud.com> | 2022-08-26 14:18:47 +0200 |
| commit | d1317e75409bdb2f37129ce07aee621bbe2b1a6c (patch) | |
| tree | 543959ed8ab7e2a961ac08dfe8631074161b4799 /apps | |
| parent | c1df72fc8b578e201b89793c65755f767f5073c7 (diff) | |
| download | nextcloud-server-d1317e75409bdb2f37129ce07aee621bbe2b1a6c.tar.gz nextcloud-server-d1317e75409bdb2f37129ce07aee621bbe2b1a6c.zip | |
Remove copy in file actions when not allowed
Signed-off-by: Vincent Petry <vincent@nextcloud.com>
Diffstat (limited to 'apps')
| -rw-r--r-- | apps/files/js/fileactions.js | 9 | ||||
| -rw-r--r-- | apps/files/js/fileinfomodel.js | 13 | ||||
| -rw-r--r-- | apps/files_sharing/src/share.js | 4 |
3 files changed, 24 insertions, 2 deletions
diff --git a/apps/files/js/fileactions.js b/apps/files/js/fileactions.js index 9b86c6521ae..f342f21a4fb 100644 --- a/apps/files/js/fileactions.js +++ b/apps/files/js/fileactions.js @@ -673,6 +673,9 @@ displayName: function(context) { var permissions = context.fileInfoModel.attributes.permissions; if (permissions & OC.PERMISSION_UPDATE) { + if (!context.fileInfoModel.canDownload()) { + return t('files', 'Move'); + } return t('files', 'Move or copy'); } return t('files', 'Copy'); @@ -685,7 +688,11 @@ var permissions = context.fileInfoModel.attributes.permissions; var actions = OC.dialogs.FILEPICKER_TYPE_COPY; if (permissions & OC.PERMISSION_UPDATE) { - actions = OC.dialogs.FILEPICKER_TYPE_COPY_MOVE; + if (!context.fileInfoModel.canDownload()) { + actions = OC.dialogs.FILEPICKER_TYPE_MOVE; + } else { + actions = OC.dialogs.FILEPICKER_TYPE_COPY_MOVE; + } } var dialogDir = context.dir; if (typeof context.fileList.dirInfo.dirLastCopiedTo !== 'undefined') { diff --git a/apps/files/js/fileinfomodel.js b/apps/files/js/fileinfomodel.js index 83a8c62592b..79575109656 100644 --- a/apps/files/js/fileinfomodel.js +++ b/apps/files/js/fileinfomodel.js @@ -126,7 +126,18 @@ }); return deferred.promise(); - } + }, + + canDownload: function() { + for (const i in this.attributes.shareAttributes) { + const attr = this.attributes.shareAttributes[i] + if (attr.scope === 'permissions' && attr.key === 'download') { + return attr.enabled + } + } + + return true + }, }); if (!OCA.Files) { diff --git a/apps/files_sharing/src/share.js b/apps/files_sharing/src/share.js index 76c007b5218..93891cbf287 100644 --- a/apps/files_sharing/src/share.js +++ b/apps/files_sharing/src/share.js @@ -94,6 +94,10 @@ import { getCapabilities } from '@nextcloud/capabilities' } if (_.isFunction(fileData.canDownload) && !fileData.canDownload()) { delete fileActions.actions.all.Download + if (fileData.permissions & OC.PERMISSION_UPDATE === 0) { + // neither move nor copy is allowed, remove the action completely + delete fileActions.actions.all.MoveCopy + } } tr.attr('data-share-permissions', sharePermissions) tr.attr('data-share-attributes', JSON.stringify(fileData.shareAttributes)) |