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

ardas’s picture

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.

ardas’s picture

... 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:

      case "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:

  $mode = arg(0);
  if ($mode == 'taxonomy' || $mode == 'taxonomy_menu') {

Thank you.

guardian’s picture

any chance to get integration commited to cvs ?

nedjo’s picture

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.

ardas’s picture

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.

Summit’s picture

Version: 4.7.x-1.x-dev » 5.x-2.x-dev

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

NancyDru’s picture

Status: Active » Postponed
NancyDru’s picture