Hello,

The attached patch allows SimpleMenu to display more than one menu (or sub-menu) in it's menu bar. When several menus are displayed, they are displayed under a drop down of the menu name.

Amongst other uses, this allows site admins to create a custom menu for each role. A given users will then see only the menus that match his or her roles (this effectively fixes the "Menu based on role" issue http://drupal.org/node/208176 ).

Thank you for this great module, hope you find this patch useful :)
Anselm

CommentFileSizeAuthor
simplemenu.patch2.79 KBAlice Heaton

Comments

Alice Heaton’s picture

Alice Heaton commented

Note that this renders the "add devel module links" option redundant, as you can simply add the devel menu. I have not however removed the option, as I think it's up to the maintainer to see how s/he best wishes to deal with that issue.

moshe weitzman’s picture

moshe weitzman commented

I agree that removing special devel handling as proposed here makes sense.

Also, time for a new release of this module. Lots of updates since December 2008.

cyoong87’s picture

cyoong87 commented
Version: 6.x-1.2 » 5.x-5.0

Does the patch work for D5?

AlexisWilke’s picture

AlexisWilke commented
Status: Needs review » Active

Hi Anselm,

Thank you for the patch! 8-)

I'm not too sure I like the concept of concatenating multiple menus. I'd rather have a weight and show just one menu as chosen.

At this time, I will hold on this patch (not apply it right now), since I was thinking to add an option for each user to select whether they see a simplemenu or not, we could offer them to choose that menu at that point. Then I'm not so sure this option really applies.

Thank you.
Alexis Wilke

cyoong87 -- we don't support version 5 anymore, also the patch in this issue is for D6 so I fixed the version. If you can submit a patch for D5, open a new issue and post the patch there. Thank you.

AlexisWilke’s picture

AlexisWilke commented
Version: 5.x-5.0 » 6.x-1.x-dev
Status: Active » Needs review
AlexisWilke’s picture

AlexisWilke commented
Assigned: Unassigned » AlexisWilke
Status: Active » Fixed

Okay. Done. I changed the loop completely based on version 2.x and I included your patch tweaked to work with the new scheme.

Thank you.
Alexis Wilke

Status: Fixed » Closed (fixed)

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