Patch to allow selecting vocabs for menu trail [#199365]

The attached patch fixes a couple of typos in menutrails.module which hide the help text on the settings form.

It also adds the ability to allow the user to select which vocabs to set menu trails for.

The method is basically to select the categories to trail and they are then evaluated on save on the settings form.

This fixes loading issues when there are a large number of vocabularies / terms

Thanks for the great module! much appeciated :-)

Comment	File	Size	Author
	menutrail.module.20071210.patch	3.35 KB	davidwhthomas

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

psynaptic CreditAttribution: psynaptic commented 29 October 2008 at 09:57

Over 10 months and not a single reply. Shame on you!

Comment #2

sun

German

Karlsruhe

CreditAttribution: sun commented 29 October 2008 at 11:08

Status:

Needs review

» Closed (won't fix)

Well, the reason is simple - this patch tried to change too many things in one fell swoop, and even worse, does not adhere to Drupal's coding standards.

I believe something like the requested feature is already available in the D6 branch of this module.

Typos are already fixed in CVS, hence marking this issue as won't fix.

Comment #3

psynaptic CreditAttribution: psynaptic commented 29 October 2008 at 12:07

Ahh ok then, makes sense.

Comment #4

davidwhthomas CreditAttribution: davidwhthomas commented 29 October 2008 at 23:17

Status:

Needs work

» Closed (won't fix)

Can you be more specific about the requirements here?

Sites with a reasonable number of vocabularies or terms may crash on the default page as the module tries to build the menu tree select box for every single term and vocab.

All the patch does is allow selecting the vocabularies to apply trailing to on the admin settings page.

It provides a select box to choose the vocabs to trail and then on save shows those vocabs in their own fieldsets with terms for trailing.

I could run it through coder or similar if there are any code style issues but it looks fine to me.

I use this patch all the time.

regards,