Integration with taxonomy_menu for Drupal 5.x

rmcmillan - March 2, 2008 - 18:47
Project:Taxonomy context
Version:5.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:duplicate
Description

This patch will allow for the display of taxonomy_context's taxonomy description in the taxonomy_menu module.

I am using taxonomy_context 5.x-1.x-dev and taxonomy_menu 5.x-1.03.

I believe older versions of taxonomy_menu will not work right with this patch.

Cheers,

Rodger

AttachmentSize
taxonomy_context_WorkWithTaxonomyMenu.patch3.15 KB

#1

lee20 - March 9, 2008 - 09:29

@rmcmillan

Thanks for the patch, but can you briefly describe what is gained by integrating with the Taxonomy Menu module?

I have only briefly used the Taxonomy Menu module so I would suspect this adds the ability to embed term links directly into the navigation menu. But after looking at the patch, it looks like it only adds the ability to display the term description on the term pages created by Taxonomy Menu. Please clarify what the patch is meant to do. As if I am correct, I think this may be better served as a feature request as under Taxonomy Menu.

#2

rmcmillan - March 18, 2008 - 04:16

@lee20

You are correct on everything. The patch allows taxonomy_context to display its information in the help section of a taxonomy_menu generated page.

I think though that the taxonomy_context module is the proper place for the code since the taxonmy_menu module does not know anything about a displaying this "context" information. The only feature is the adding of the taxonomies into the menu, but there is not much in the way of generating any information about it.

I am admittedly lazy - this was the easiest way to accomplish showing the taxonomy_context info in taxonomy_menu :)

I do understand why you would not want this in the code trunk though since you would then have to update the code as the taxonomy_menu module changes. I had this code working a couple months back - and then I updated the taxonomy_menu module and it broke.

Perhaps with 6.x now, this is not a concern?

#3

lee20 - March 20, 2008 - 05:22

@rmcmillan

I think though that the taxonomy_context module is the proper place for the code since the taxonmy_menu module does not know anything about a displaying this "context" information.

The reason I think this belongs in Taxonomy Menu and not in TaxCon is primarily because TaxMenu creates it's own pages for displaying taxonomy terms. It is a relatively small feature, and is not menu related but I feel the module should provide the options for it's pages and would then be available without needing TaxCon to achieve something so small. I don't mean to beat a dead horse.....just wanted to share my reasoning.

Perhaps with 6.x now, this is not a concern?

I haven't worked with D6 yet :( But I hear the menuing system is much improved; not sure on specifics though and I guess maybe they were referring to the menu administration which has a sweet UI in D6.

#4

lee20 - March 20, 2008 - 05:42

Ooops. Duplicate entry.

#5

Summit - September 21, 2008 - 21:00

Subscribing, greetings, Martijn

#6

jenlampton - November 10, 2008 - 05:38

I would officially like to request the feature you thought this was... the ability to embed contextual term links directly into the D5 menu system. I think the "navigation" menu might not be the best place, but at least it's a start.

Modules like nice_menus rely on menus being on the menu system, so if we could move this module in that direction I would be one happy camper :-)

Jen

#7

NancyDru - February 26, 2009 - 18:58
Status:needs work» duplicate

#71989: Integration with taxonomy_menu

 
 

Drupal is a registered trademark of Dries Buytaert.