fix: implement 'per user inherit' logic for folder delete permissions check #3404
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
To test:
occ config:app:set groupfolders acl-inherit-per-user --value true
G1
andG2
U1
that is a member ofG1
andG2
F1
with full access forG1
,G2
andadmin
and enable ACL for the groupfolderF1/Test
F1
which sets all permissions forG1
to allowF1/Test
which sets the delete forG2
to denyAt this point the output of
occ groupfolders:permissions <folder id>
should look like:U1
try to deleteF1/Test
Current behavior
Even though
U1
sees the delete option, trying to delete the folder failsNew behavior
U1
can successfully delete the folder.