The enable/disable admin modules functionality is great but it's a shame that its hard-coded. A simple drupal_alter() would greatly enhance this and allow contribs to specify whether they or any of their submodules should be considered "admin modules".
| Comment | File | Size | Author |
|---|---|---|---|
| #3 | 1635228-3.patch | 612 bytes | areke |
| #1 | admin_menu-developer-modules-alter-1635228-1.patch | 593 bytes | alexweber |
Comments
Comment #1
alexweber commentedAttached is a patch that implements this.
The next step is probably getting contribs to implements this and gradually reduce the hard-coded list, but that's another issue... :)
Comment #2
areke commentedadmin_menu.inc was removed from the module, and this code does not exist anywhere else. This is no longer an issue.
Comment #3
areke commentedOops, I made a mistake. The patch actually works fine; however,
drupal_alter()is deprecated in 8.x.\Drupal::moduleHandler()->alter()should be used instead.Here's an updated patch.
Comment #4
alexweber commentedThis would be really cool + a backport too...
Comment #5
jibranI have created a fork of admin_menu 8.x-3.x on github. Please help me fix it for Drupal 8
PRs are welcome. https://github.com/jibran/admin_menu
Comment #6
alexweber commented@jibran, sounds like a better idea! Are the 8.x issues getting moved over by any chance?
Comment #7
jibranMy plan is to first get it up and running again then we can talk about the issues.
Comment #8
dave reidThe 'disable development modules' functionality has been removed as of #2392519: Remove 'Disable Development modules'.
Comment #9
alexweber commentedThat's too bad, although it's nice to have some closure for this issue... :)
edit: in hindsight, good call removing that functionality though