Flowpack / media-ui

The development repository for the new Neos media management interface
GNU General Public License v3.0
21 stars 16 forks source link

Prevent adding assets to "all" if user has only privilege to see some collections. #246

Open Sebobo opened 3 hours ago

Sebobo commented 3 hours ago

Description

Currently when a user adds an asset to the virtual "all" collection, meaning no collection, but can only see a certain set of collections, the asset is not visible to them after upload.

Steps to Reproduce

  1. Restrict role to only see one collection
  2. Upload asset to "all"

Expected behavior

The asset goes to their first accessible collection or they cannot select "all".

Actual behavior

The asset disappears

Sebobo commented 3 hours ago

@lorenzulrich do you think we should hide/disable the "All" collection or change its behaviour to match the users privileges?

lorenzulrich commented 2 hours ago

I'm not sure if there is a "one size fits all approach" to this.

Some considerations

Currently, I'd be in favour of having a privilege for the "All" entry point so I could leave it on for an admin if necessary, but deactivate it for editors.