hi there - nice module!
but it would be even better, if one could choose the menus to be displayed on one page together (like a complete sitemap)
i did not find any module to combine menus to one either...
greetings
sebastian

CommentFileSizeAuthor
#9 menuall.patch6.21 KBeabrand
#7 menuall.patch6.41 KBeabrand
#6 menuall.patch6.88 KBeabrand

Comments

Crell’s picture

Version: 5.x-1.x-dev » 6.x-1.x-dev
Status: Active » Postponed (maintainer needs more info)

New features will only go into the 6.x version.

I am not entirely clear what the use case here is. I'm not against it, I suppose, but I'm curious why it would be needed.

Anonymous’s picture

sitemap/all works great in dev version, thanks.

Crell’s picture

Yeah, I had a client ask for that at some point. It needs to be ported forward to the D6 version, then both need a stable release to include it. Anyone want to do the D6 port of that feature? :-)

Anonymous’s picture

+1 for this functionality!
Then it will make it a great sitemap.

@msn
What do you mean with 'sitemap/all' ?

Anonymous’s picture

JoepH:

Oehh...., What I remember.
The sitemap/all issue is outdated. In an older version you could not create a complete sitemap, only the dev version (at that time) owned this feature. It this moment the dev version is an official release.

With compliments to the module developer(s).

Martin

eabrand’s picture

StatusFileSize
new6.88 KB

Here's a patch for D6

eabrand’s picture

Status: Postponed (maintainer needs more info) » Needs review
StatusFileSize
new6.41 KB

I have an update to the previous patch

Crell’s picture

Status: Needs review » Needs work

- It looks like the patch has tabs in it instead of two-spaces for indentation. All Drupal code should use two-space indentation.

- The indentation is off in menutree_display_all().

- The default weight for menus in the "all" page is set to none, so no menus will show until the admin page has been submitted. However, the defaults on the admin page are to show, in increasing order. That creates a disconnect if you're just looking at the admin page but haven't submitted it. We should probably normalize those to none in both cases for consistency.

- The if-else in menutree_display() should have the else on its own line:

if ($blah) {

}
else {

}

- I'm unclear why you're switching the constants from a bitwise check to an equality check.

eabrand’s picture

Assigned: Unassigned » eabrand
Status: Needs work » Needs review
StatusFileSize
new6.21 KB

Here is the updated patch

Crell’s picture

Status: Needs review » Fixed

Committed #9. Awesome!

Status: Fixed » Closed (fixed)

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