- πΊπΈUnited States alfattal Minnesota
The MR in #2 and patch in #3 works fine. However, I have a use case where a role with the permission to "Delete any files" needs to have access to the "Unmanaged File List" to perform cleaning, but this role should NOT have access to the "Unmanaged File Settings".
The current patch removes access to both. Hence, we need more granular permissions to limit the access to the settings separately from the access to the list.
- Status changed to Needs review
over 1 year ago 4:21pm 18 April 2023 - π§πͺBelgium daften
A new patch file with separate permissions.
IMO the unmanaged files list could be put in the content menu, but that's a separate issue if other people agree it would be better.
- Status changed to Needs work
over 1 year ago 2:57pm 4 May 2023 - πΊπΈUnited States alfattal Minnesota
@daften I've applied your patch in #6, went to the permissions and granted a specific role the access to administer tasks only but seems like nothing has changed. That role and every role still have access to Administer unmanaged files and Administer unmanaged files settings. For this reason, I'm changing the status to Needs work.
- Status changed to Needs review
over 1 year ago 1:31pm 25 May 2023 - π³π΄Norway vegardjo
Hi, I can't reproduce #7.
I have tested this with 3 different roles, one that has only the "administer unmanaged files", one that has only the "administer unmanaged files settings" and one that has none, and it works like expected. I had to clear cache after applying the patch, though.
- Status changed to RTBC
over 1 year ago 6:42pm 1 June 2023 - πΊπΈUnited States alfattal Minnesota
@vegardjo Thank you for the cache clearing tip. I've made another test for the patch in #6 and after clearing the cache, I can confirm that it's working as expected. Moving this to RTBC!