Out of curiosity, I ran a drupal 7 conversion on a drupal 6 system. Also then installed latest 7.x. simplemenu.

Even though simplemenu works for Admin menu, it does not seem to work for my old D6 menu. I get the following warning:

Notice: Undefined index: menu_item_link in simplemenu_inactive_parents_theme_registry_alter() (line 30 of ...\sites\all\modules\simplemenu\simplemenu_inactive_parents.module).

Any thoughts?

Comments

Anonymous’s picture

Anonymous (not verified) commented

I'm getting a similar error message, for a Drupal 7 installation:

Notice: Undefined index: menu_item_link in simplemenu_inactive_parents_theme_registry_alter() (line 29 of /var/www/drupal7/modules/simplemenu/simplemenu_inactive_parents.module).

robmalon’s picture

robmalon commented

See the comment at the end of: http://api.drupal.org/api/drupal/includes--menu.inc/function/theme_menu_...

That theme hook appears to have changed to http://api.drupal.org/api/drupal/includes--menu.inc/function/theme_menu_... in Drupal 7.

chalee’s picture

chalee commented

I'm having same problem. Has this issue been fixed in dev version or a patch done yet.

AlexisWilke’s picture

AlexisWilke commented
Status: Active » Postponed

The D7 version is not ready for prime time yet. It only displays one of the system menus and the name of the menu is hard coded...

Unfortunately I do not find the time to work on this module for D7...

I'd be happy to give someone maintainer-ship for D7 though.

Thank you.
Alexis Wilke

crobinson’s picture

crobinson commented

I can do some work on it if you want. I plan to use it in a new project.

AlexisWilke’s picture

AlexisWilke commented

crobinson,

Be my guest! I actually turned off my D7 version... I just don't have the time to work on that.

Thank you.
Alexis

P.S. I gave you access so you can access the git in write mode and create new releases (once the D7 version is usable...)

crobinson’s picture

crobinson commented
Assigned: Unassigned » crobinson
Category: support » bug
Status: Postponed » Needs review

fe56961 changes theme_menu_item_link to theme_menu_link, which resolves the error on my test site. But I don't actually use this module so I don't have a good test to see if it solves the entire problem. Would somebody please test and comment?

If it's still not working I'll need better menu test data to work with. It would help a lot if somebody would be willing to export their menu_* items from a site that is experiencing the problem, with a detailed description of how to reproduce the problem.

manu manu’s picture

manu manu
Primary language French
Location Aix-en-Provence
commented
Status: Needs review » Reviewed & tested by the community

Worked for me.

Here is a link to the patch for people willing to test this fix:
http://drupalcode.org/project/simplemenu.git/patch/fe569614ab286460d57e9...

crobinson’s picture

crobinson commented

Thank you for submitting this patch. I'll review it this evening.

crobinson’s picture

crobinson commented
Status: Reviewed & tested by the community » Fixed

OK, I'm a space cadet tonight, I didn't realize this was _my_ patch. It looks like you're confirming that this patch does in fact resolve the issue, so I'm marking the ticket for closure in 2 weeks if there's no other comment.

Status: Fixed » Closed (fixed)

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