We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Permission Issue - there is a missing check for the 'D' on files to validate whether the Download option is shown in the File Sidebar.
extendedAttributes.push({ scope: 'permissions', key: 'download', enabled: false })
Should not show 'Download' under actions.
Show 'Download' under actions.
Change this function:
web/packages/web-app-files/src/helpers/resources.js
Lines 63 to 66 in 589cf7e
To this:
canDownload: function() { return !isFolder && this.permissions.indexOf('R') >= 0 },
I have tested this on our own codebase with success.
See #5452 for a similar related issue for copying of files.
The text was updated successfully, but these errors were encountered:
oCIS has an X webdav permission now, which prohibits downloads. Was implemented in web with #10973
X
Sorry, something went wrong.
Successfully merging a pull request may close this issue.
Permission Issue - there is a missing check for the 'D' on files to validate whether the Download option is shown in the File Sidebar.
Steps to reproduce
extendedAttributes.push({ scope: 'permissions', key: 'download', enabled: false })
Expected behaviour
Should not show 'Download' under actions.
Actual behaviour
Show 'Download' under actions.
Recommended Fix
Change this function:
web/packages/web-app-files/src/helpers/resources.js
Lines 63 to 66 in 589cf7e
To this:
I have tested this on our own codebase with success.
See #5452 for a similar related issue for copying of files.
The text was updated successfully, but these errors were encountered: