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.

CommentFileSizeAuthor
#1 admin_menu_614730.patch445 bytespflame
admin_menu_problem.png109.05 KBpflame

Comments

pflame’s picture

pflame commented
Status: Active » Needs review
StatusFileSize
new admin_menu_614730.patch445 bytes

I fixed this by providing 'render element' attribute while defining admin_menu_links theme. I attached patch to this comment.

jurgenhaas’s picture

jurgenhaas
he/him
Primary language German
Location Gottmadingen
commented

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.

JamesAn’s picture

JamesAn commented

Same problem here. Patch works wonderfully. Another +1 review. RBTC soon?

sun’s picture

sun
Primary language German
Location Karlsruhe
commented
Component: User interface » Code
Status: Needs review » Fixed

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.

Status: Fixed » Closed (fixed)

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