admin menu is not appearing after I enable it

azriprajwala - October 26, 2009 - 12:48
Project:Administration menu
Version:7.x-3.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed
Description

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.

AttachmentSize
admin_menu_problem.png109.05 KB

#1

azriprajwala - October 27, 2009 - 08:15
Status:active» needs review

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

AttachmentSize
admin_menu_614730.patch 445 bytes

#2

jurgenhaas - October 27, 2009 - 08:47

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

JamesAn - October 28, 2009 - 05:19

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

#4

sun - October 28, 2009 - 13:29
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.

#5

System Message - November 11, 2009 - 13:30
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.