Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
The toolbar shows a link to "Workflow" under configuration even when there is nothing there. The system module ships with the Workflow menu link in system.links.menu.yml - it only shows up on /admin/config if there is a module that adds to the menu (for example Workflows) but for some reason it is in the toolbar anyway.
To reproduce install the standard install profile and put the toolbar in vertical mode.
Click on the link takes you to admin/config/workflow where it tells you that you don't have any administrative items.
Proposed resolution
Fix the toolbar module to use the same rules as admin/config
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#16 | 2850533-after_patch-14.png | 242.43 KB | Abhijith S |
#16 | 2850533-before_patch-14.png | 469.14 KB | Abhijith S |
#14 | 2850533-14.patch | 3.57 KB | smustgrave |
| |||
#14 | 2850533-14-tests-only.patch | 1.11 KB | smustgrave |
Screen Shot 2017-02-07 at 18.25.23.png | 55.61 KB | alexpott |
Comments
Comment #2
alexpottComment #3
Anonymous (not verified) CreditAttribution: Anonymous commentedA similar problem occurs when there is a user role with the system permissions:
Comment #14
smustgrave CreditAttribution: smustgrave at Mobomo commentedThis came up on as a daily bug smash target
I confirmed this is an issue in D9.5 but only seems to be an issue for admins from what I can tell.
Moved the menu link and route from system to workflows module.
Added a simple test but not sure if there is a better spot.
Comment #16
Abhijith S CreditAttribution: Abhijith S as a volunteer and at Srijan | A Material+ Company for Drupal India Association commentedApplied patch #14 on 9.5.x and it works.The Workflow menu link is not appearing in the configuration menu after applying this patch.It only appears when the Workflows module get installed.
Attaching screenshots below.
Before patch:
After patch:
Comment #18
smustgrave CreditAttribution: smustgrave at Mobomo commentedRunning for 10.1 as this one is just a small annoyance
Comment #19
catchThis is a duplicate of #296693: Restrict access to empty top level administration pages.