Will there be a version for Drupal 7?

CommentFileSizeAuthor
#4 simplemenu-791664.zip3.61 KBdtengeri

Comments

AlexisWilke’s picture

When do you send me a patch? 8-))

AlexisWilke’s picture

Title: Drupal 7 Version » Drupal 7 Version started...
Version: 6.x-2.x-dev » 7.x-1.x-dev
Assigned: Unassigned » AlexisWilke

Okay... it's in progress... I have the simplemenu module that kind of work. If you want to give it a try... (7.x-1.x-dev should appear within 12h)

Help welcome 8-)

The menu functions are completely different so I had to rewrite quite a bit of the code to make it work and at this point the menu is forced to "management".

Thank you.
Alexis Wilke

Anonymous’s picture

Thanks @AlexisWilke! I'll be glad to help when my time opens up...

dtengeri’s picture

StatusFileSize
new3.61 KB

Hi!

I've done some porting based on the latest 7-dev version.
What's working:

  • simpletest.module: now you can select any menu to display
    • I've rewritten the simplemenu_tree_all_data() function, based on menu_tree_all_data() and the Drupal 6 version of simplemenu_tree_all_data().
      Please check this, because i'm not a master of the menu system and its caching mechanism.
    • Fixed hook_permission()
    • fixed some codig style issues, found by the Coder module.
  • simpletest_devel.module

What's not working with this patch:

  • simplemenu_multi_menu
  • simplemenu_inactive_parents

These two change the menu_item_link's theme function, but there isn't a theme_menu_item_link in Drupal 7, so this needs more time.

crobinson’s picture

Assigned: AlexisWilke » crobinson
Status: Active » Fixed

2801bf9 is a manual merge of the patch provided by dtengeri, thanks for the contribution. I had to do this by hand because the project has apparently evolved since you supplied the original patch. However, it looks like there were still things the patch provided that hadn't been addressed.

I've tested this in my own installation and it appears to work well. Please let me know if you have any issues, as I'm not using some of the additional SimpleMenu add-ons in the project.

Status: Fixed » Closed (fixed)

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