Posted by pflame on October 26, 2009 at 12:48pm
| Project: | Administration menu |
| Version: | 7.x-3.x-dev |
| Component: | Code |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
I am using drupal7 from head, I just did cvs checkout from the drupal head. When I enable admin-menu module I saw these notices and warnings. And the menu is not displaying.
* Notice: Undefined index: render element in theme() (line 803 of /var/www/drupal-reviews/drupal-cvs/includes/theme.inc).
* Notice: Undefined index: elements in theme_admin_menu_links() (line 486 of /var/www/drupal-reviews/drupal-cvs/sites/all/modules/admin_menu/admin_menu.module).
* Warning: Invalid argument supplied for foreach() in element_children() (line 5044 of /var/www/drupal-reviews/drupal-cvs/includes/common.inc).
* Notice: Undefined index: render element in theme() (line 803 of /var/www/drupal-reviews/drupal-cvs/includes/theme.inc).
* Notice: Undefined index: elements in theme_admin_menu_links() (line 486 of /var/www/drupal-reviews/drupal-cvs/sites/all/modules/admin_menu/admin_menu.module).
* Warning: Invalid argument supplied for foreach() in element_children() (line 5044 of /var/www/drupal-reviews/drupal-cvs/includes/common.inc).
* Notice: Undefined index: render element in theme() (line 803 of /var/www/drupal-reviews/drupal-cvs/includes/theme.inc).
* Notice: Undefined index: elements in theme_admin_menu_links() (line 486 of /var/www/drupal-reviews/drupal-cvs/sites/all/modules/admin_menu/admin_menu.module).
* Warning: Invalid argument supplied for foreach() in element_children() (line 5044 of /var/www/drupal-reviews/drupal-cvs/includes/common.inc).When i enable Administration menu Toolbar style module , the theme header screwed up, please see the attached document.
| Attachment | Size | Status | Test result | Operations |
|---|---|---|---|---|
| admin_menu_problem.png | 109.05 KB | Ignored: Check issue status. | None | None |
Comments
#1
I fixed this by providing 'render element' attribute while defining admin_menu_links theme. I attached patch to this comment.
#2
Had the same effect, your patch works nicely. Thanks.
I had modified 'arguments' into 'variables' as that was another API change in the latest D7 release, but with no effect at all. So yours is good help.
#3
Same problem here. Patch works wonderfully. Another +1 review. RBTC soon?
#4
Thanks for reporting, reviewing, and testing! Committed to all branches.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.
#5
Automatically closed -- issue fixed for 2 weeks with no activity.