Skip to content
New issue

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

[media] fixes permissions. Allows media_read to download, renames "Delete" to "Hide". #9222

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

racostas
Copy link
Contributor

Brief summary of changes

Allows the media_read permission to actually download (see) the files. The permission in this descriptions stats Media: View/Download Candidate Media Files but was not actually allowing to download (see) the documents.

media_write was substituted by media_read in the corresponding cases.

The permission Media: Edit/Upload/Delete Candidate Media Files is renamed to Media: Edit/Upload/Hide Candidate Media Files since we are not able to Delete files but to Hide they.

  • Have you updated related documentation?

Testing instructions (if applicable)

  1. Go to the media module with a user having only the media_read permission.
  2. You should be now capable of downloading (seeing) the files.
  3. As admin go to edit candidate and see the permission Media: Edit/Upload/Delete Candidate Media Files is renamed to Media: Edit/Upload/Hide Candidate Media Files .
  4. The rest of the functionalities in the module should be rest unchangeable.

Link(s) to related issue(s)

@racostas racostas added 26.0.0-bugs Issues that were raised during the release testing for 26.0.0 Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) SQL PR contains SQL modifications such as schema changes and new SQL scripts labels Apr 25, 2024
@@ -33,7 +33,7 @@ CREATE TABLE `permissions` (
'View/Create/Edit',
'Create/Edit',
'Edit/Upload',
'Edit/Upload/Delete'),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we want to change this? This is the generic category used for all module permissions.

@ridz1208

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@racostas @driusan I'm okai with changing it if it's the only module using it.

Or, to be safe, just create a new one

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
26.0.0-bugs Issues that were raised during the release testing for 26.0.0 Bug PR or issue introducing/requiring bug fixes (not mutually exclusive with the Feature label) SQL PR contains SQL modifications such as schema changes and new SQL scripts
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[media] Permission names need revising
4 participants