Add alter hook for links generated by Extra Links deriver

Adds hook_admin_toolbar_extra_links_alter()

I would very much appreciate this addition. The ExtraLinks class unfortunately adds some links which are unncessary and duplicate, and I haven't found a built-in way to remove them.

It works with the patch #2 and a hook like this:

/**
* Implements hook_admin_toolbar_extra_links_alter().
*/
function mymodule_admin_toolbar_extra_links_alter(&$links): void {
unset($links['media_library']);
}

I guess for this to be merged we need a test, right?

Prashant.c → made their first commit to this issue’s fork.

Thanks @alieffring and @Ammaletu.
Pushed the code by creating a issue fork, some documentation work is required in admin_toolbar_tools/admin_toolbar_tools.api.php .

I corrected the doc block in the api file. There is currently only one test in this module, so I'll be bold here and set this to "needs review". :-)

Thanks a lot everyone for raising this issue and the code contributions, it's greatly appreciated!

It would seem the initial problem:

Enable the admin_toolbar_tools submodule. It generates a bunch of links, but you can't change them.

could already be achieved by using the approach detailed in comment #3488920-2: Change the deriver class of a menu link → , with an implementation of hook_menu_links_discovered_alter.

Could you please try the suggested approach and let us know if this could work?

For the time being, closing this issue as Won't fix, since it seems there is already a proper solution provided by Drupal Core.

Feel free to re-open the issue if you consider these changes would still be needed, we would be glad to take another look.

Please let us know if you have any questions on this comment, this issue or the project in general, we would surely be glad to help.
Thanks again for your interest in the module and all your contributions! 🙂

Hi @dydave!

Thanks for pointing out the solution from the other issue. That indeed works as expected, and I don't know why I didn't find that hook before.

Here is how I remove a link from the menu now:

/**
 * Implements hook_menu_links_discovered_alter().
 */
function MY_MODULE_menu_links_discovered_alter(&$links): void {
  // Content > Media already links to the same page, so we don't need Content >
  // Media > Media library.
  unset($links['admin_toolbar_tools.extra_links:media_library']);
}

Thanks a lot Johannes (@ammaletu) for the positive feedback, it's greatly appreciated!

This will certainly help others with the same type of requirements and should be very helpful to us as an example when documenting related issue 💬 Change the deriver class of a menu link Active .

Glad this could work for you 🙂
Thanks again for reporting back and documenting your changes! 🙏