It would be nice if I could easily get to all the menu items in the 'devel' menu like theme registry, enter PHP, etc, from admin_menu.

#3 admin_menu.devel_.3.patch902 bytessun
PASSED: [[SimpleTest]]: [MySQL] 228 pass(es). View
#3 interdiff.txt711 bytessun
#1 1899474-devel-entire-menu.patch1.11 KBDave Reid
PASSED: [[SimpleTest]]: [MySQL] 228 pass(es). View
#1 Selection_028.png23.82 KBDave Reid
Members fund testing for the Drupal project. Drupal Association Learn more


Dave Reid’s picture

Status: Active » Needs review
23.82 KB
1.11 KB
PASSED: [[SimpleTest]]: [MySQL] 228 pass(es). View

Patch with screenshot attached. It degrades nicely if you don't have access to any devel menu items, hiding the top-level 'Devel' item.

Dave Reid’s picture

Personally I think this a much better solution than hard-coding one devel item, with a hard-coded permission string. Let me know what you think sun!

sun’s picture

711 bytes
902 bytes
PASSED: [[SimpleTest]]: [MySQL] 228 pass(es). View

Makes sense to me. Now that I see it, I wondered why we didn't do this earlier. ;)

Only minimal adjustments:

- Make the new 'Development' item come after 'Flush all caches', as before.

- Leverage menu_link_tree() like we do for injecting the node/add tree.

sun’s picture

Status: Needs review » Fixed

Thanks for reporting, reviewing, and testing! Committed to 7.x-3.x and 8.x-3.x.

(Forgot to update my git index prior to commit, so #3 was a separate follow-up commit... ;))

A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.

jibran’s picture

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.