Problem/Motivation
See core issue: #1869638: Make the menu shown in the administration menu tray configurable
The patch in that issue adds a new admin_tray.menu_name variable in toolbar.settings config, for configuring which menu gets used in the toolbar administration tray.
Proposed resolution
The core patch makes a change to ToolbarController::preRenderAdministrationTray() to use a new admin_tray.menu_name variable in toolbar.settings config. With this variable and the Drupal configuration override functionality the amswap module codebase would be much more cleaner, and more compatible with other modules and themes.
User interface changes
none
API changes
The menu pre render call is removed.
Configuration override is used to replace the menu.
Data model changes
none
Comments
Comment #2
pasquallePatch attached. Postponing as dependent on #1869638
Comment #3
droces commentedYes, that core patch would be great, and would greatly simplify this module. But it doesn't look like it's going to happen any time soon though...
Comment #4
ricovandevin commentedThe patch does not apply anymore in version 3.1.0.
Comment #5
arrowRerolled for 3.1.0 and added check to allow fallback to existing behavior.
Comment #6
klonos...linkifying the core issue number in the issue summary.
Comment #7
cedeweyComment #8
pasqualleThe patch does not apply
Comment #9
pasquallereroll