Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
After unchecking "Cache menu in client-side browser", checking a few Developer modules to keep enabled and then submitting, I'm getting this error on the admin pages for admin_menu:
Notice: Undefined index: name in admin_menu_theme_settings() (line 708 of /.../sites/all/modules/admin_menu/admin_menu.inc).
This is on a fresh install with only a few modules enabled outside of core like views, ctools, fontyourface.
Comment | File | Size | Author |
---|---|---|---|
#3 | admin_menu-1997386.patch | 555 bytes | Sweetchuck |
Comments
Comment #1
BWPanda CreditAttribution: BWPanda commentedGetting the same error.
Discovered it's referring to the 'bad' sub-module of Coder. Similar issues relating to the missing name value were addressed by the Coder team by adding
hidden = TRUE
to bad.info.Admin Menu, however, still displays hidden modules; hence the error. I recommend Admin Menu does something like what system_modules() does and ignores hidden modules:
Comment #2
kanenas CreditAttribution: kanenas commentedI am experiencing this "Notice"...
Notice: Undefined index: name in admin_menu_theme_settings() (line 708 of /path/to/file/sites/all/modules/admin_menu/admin_menu.inc).
just by accessing "admin/config/administration/admin_menu".
Comment #3
SweetchuckComment #4
Alan D. CreditAttribution: Alan D. commentedLooks good to me, I was about to write a patch for the same notice ;)
Comment #5
mfuller526 CreditAttribution: mfuller526 commented3: admin_menu-1997386.patch queued for re-testing.
Comment #6
mfuller526 CreditAttribution: mfuller526 commentedUser error: patch AOK!
Comment #7
jfrederick CreditAttribution: jfrederick commented#3 worked for me.
Comment #8
euk CreditAttribution: euk commented#3 worked for me as well!
Comment #9
gngn CreditAttribution: gngn commented#3 worked for me as well, thank you!
Running the patch and checking what it actually does, I noticed that on my system there are more than 80 modules skipped because they are set to "hidden".
The array $all_modules is just used to find the "devel modules" with array_intersect_key().
Since array_intersect_key() just uses the keys, we could also:
isset($info['name'])
$all_modules[$module->name] = $module->name;
if it's not.That way (again on my system) only one module is skipped - the above mentioned 'bad' sub-module of Coder.
What do you think? I could supply a patch if anyone's interested.
Comment #10
euk CreditAttribution: euk commentedI may be a good thing.
Give us a patch )
Comment #11
Dave ReidThe 'disable development modules' functionality has been removed as of #2392519: Remove 'Disable Development modules'.
Comment #12
Alan D. CreditAttribution: Alan D. commentedThird time checking this, this should be in the latest RC version.
Issue #2392519 was committed on the 16th Dec, a few days before 7.x-3.0-rc5 (19th)