Posted by ardas on July 3, 2006 at 11:10am
Jump to:
| Project: | Taxonomy context |
| Version: | 5.x-2.x-dev |
| Component: | Code |
| Category: | feature request |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | postponed |
Issue Summary
Greetings,
As I am a big fan of such modules as taxonomy, taxonomy_context and taxonomy_menu I'd like to propose you a new feature - output taxonomy context on 'taxonomy_menu' pages as it was done for 'taxonomy/term' pages.
I found that taxonomy_menu module is very useful for e-commerce sites - it is a single normal way to display taxonomy structure as a menu which is a greate feature to display product catalogs. The only problem - there is no way to display subterms for taxonomy_menu pages.
I propose to add this feature to give taxonomy_menu users an ability to browse pages exactly like taxonomy/term does.
Thank you.
Comments
#1
Here is the result of my analysis:
"taxonomy_menu/XXXX" pages are similar to "term/XXXX" - they display nodes of the chosen term. Taxonomy_menu module has 2 advantages:
1. It can display nodes from child terms (descendants)
2. It can output terms structure in a menu block which extremely necessary for e-commerce sites when you want to have a menu of product categories
But, it also has 2 disadvantages:
1. There was no integration with nodewords (I've written a proposal, it is fixed already)
2. There is no integration with taxonomy_context module
I hope this improvement will be useful for many developers.
It gives a power of taxonomy_menu, taxonomy_context and nodewords modules combination. E-commerce site developers will understand me.
Thank you.
#2
... and here are my fixes I did in 4.6 ... I saw that 4.7 goes far away but the idea is the same:
function taxonomy_context_get_context()
Add the following case into switch:
<?phpcase "taxonomy_menu":
$args = explode('/', $_GET['q']);
$context["tid"] = array_pop($args);
$context["root_tid"] = taxonomy_context_get_top_parent($context["tid"]);
$context["vid"] = taxonomy_context_get_term_vocab($context["tid"]);
break;
?>
AND
function taxonomy_context_help($section)
Change if (arg(0) == 'taxonomy') to:
<?php$mode = arg(0);
if ($mode == 'taxonomy' || $mode == 'taxonomy_menu') {
?>
Thank you.
#3
any chance to get integration commited to cvs ?
#4
I like the idea but haven't had a chance to test it (I've never used taxonomy_menu). I would welcome a patch for the 4.7 version.
#5
I've never been working with nodewords in 4.7 yet, but I will very soon. I hope to provide you this patch when understand nodewords 4.7 differences.
#6
Hi,
I think this is still happening in drupal 5? Could this be the case?
Anyone for a patch?
Thanks a lotnin advance!
greetings,
Martijn
#7
Marked #155938: Replacing taxonomy menu? as duplicate
#8
Marked #229118: Integration with taxonomy_menu for Drupal 5.x as duplicate