For some custom requirements we need to alter the link state for the media library menu items. To do so, we have created a decorator for "media_library.ui_builder" service and swapped out buildMediaTypeMenu method (as it contains the create of the link state).
For our requirements, we don't want to change the how the menu is build but to change the link state for the menu items.
It would be cleaner to do split the create of the link state in another method, so that, sites can override the link state method only.
Move the link state creation to new method.
Needs work
10.0 ✨
Last updated
Used to track the progress of issues reviewed by the Drupal Needs Review Queue Initiative.
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.
A change record needs to be drafted before an issue is committed. Note: Change records used to be called change notifications.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request → as a guide.
createMediaTypeMenuItemLinkState
1. should be typehinted
2. Will need a test case to show this new feature in action.
3. A change record to announce the change and how it can be overridden.
Thanks!