- Issue created by @aleix
- 🇪🇸Spain aleix
Note that media library checks itself in Drupal\media_library\MediaLibraryUiBuilder::checkAccess that user has 'view media' permission and also that has createAccess of bundle that has opened the media library.
So, despite the aparently fine forbid of "Create article content" in/admin/people/permissions
and allow of "Entity: Add element article entities" in/admin/group/types/manage/GROUPTYPE/permissions
, esults in allowing the content creation,
the medialibrary accesshandler check will forbid adding media.So both general and group permissions must be given.
- 🇧🇪Belgium msnassar
Hello @aleix did you have a look into Group media library extra → module. It comes with pluggable solution for changing the behavior of the media library view when it is being opened from a group context.
The module depends on Group media library → which solves other issues when using media library with groups. Including 🐛 Incorrect Access Check on Media Library RTBC - 🇪🇸Spain aleix
Thank's @msnassar , Yes! looks promising, nice work! kudos for your contribution. (btw are you planning to request merging it with group media?)
For my use case, I need a basic functionality of media library by now, but I think that if a project requires something more powerful, it will require the use of your modules ( Or going to entity browser).
But anyway, the docs of this module also needs to be there and matching the use case of nowadays group module requirements.
Again, thank's for spending all the time coding there!
- 🇧🇪Belgium msnassar
@aleix There is no plan to add the modules mentioned in #5 to group media. This is because, they can be used without the group media module. Both has submodules to integrate with group media.
- 🇩🇪Germany vistree
@aleix,
should your hook work with group 3.3.x version? For me it seems that the group ID is not available within media library ...
I am using:
- group: 3.3.0
- group media: 4.0.3
- media_library: 10.3.8