search

silverstripe / silverstripe-assets

Silverstripe Assets component
BSD 3-Clause "New" or "Revised" License
9 stars 65 forks source link

add check for specific user inherited permission #556

Closed andrewandante closed 1 year ago

andrewandante commented 1 year ago

Connected to https://github.com/silverstripe/silverstripe-framework/pull/10819

Allows file/folder access to be restricted on a user-by-user basis, rather than having to use groups

Issue

andrewandante commented 1 year ago

can you foresee any problems if we implemented that?

Hmm. It makes "changing back" pretty tricky, which is a bit annoying. I'd prefer to go with the "hide what is irrelevant" approach - though I could also see a "Reset to defaults"-style action which pushes everything back to Inherit and clears all the relationships. I agree it looks a bit.. mediocre 😅

andrewandante commented 1 year ago

@GuySartorelli presumably this needs an updated silverstripe/framework: ^5.1 constraint too?

GuySartorelli commented 1 year ago

Ahh, yes please! I thought I checked it for that but apparently I didn't 😅