nextcloud / groupfolders

๐Ÿ“๐Ÿ‘ฉโ€๐Ÿ‘ฉโ€๐Ÿ‘งโ€๐Ÿ‘ฆ Admin-configured folders shared by everyone in a group. https://github.com/nextcloud-releases/groupfolders
https://apps.nextcloud.com/apps/groupfolders
291 stars 87 forks source link

fix: implement 'per user inherit' logic for folder delete permissions check #3404

Closed icewind1991 closed 3 days ago

icewind1991 commented 2 weeks ago

To test:

At this point the output of occ groupfolders:permissions <folder id> should look like:

+------+------------+-----------------------------------------+
| Path | User/Group | Permissions                             |
+------+------------+-----------------------------------------+
| /    | group: G1  | +read, +write, +create, +delete, +share |
| Test | group: G2  | -delete                                 |
+------+------------+-----------------------------------------+

Current behavior

Even though U1 sees the delete option, trying to delete the folder fails

New behavior

U1 can successfully delete the folder.

icewind1991 commented 2 weeks ago

/backport to stable30

icewind1991 commented 2 weeks ago

/backport to stable29