Project:Taxonomy Breadcrumb
Version:6.x-1.0
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:postponed

Issue Summary

This has been brought up before I know, I found a patch that had been written for an older version of this module. I tried to merge it into the 6.x-1.0 version with no luck.

If I enable the taxonomy_term (default) view the breadcrumbs stop working, if I disable it, they start working again. This is just for the term list pages, they semi work on the node pages.

This is a really handy module, it would be brilliant if it worked well together with Views.

Comments

#1

Category:bug report» feature request
Status:active» postponed (maintainer needs more info)

Post the patch that you are working on and I'll be happy to help you get it working.

Thanks

#2

The patch at the bottom of this page.

http://drupal.org/node/145231

It was written for Drupal 6.x though so I don't know how relevant it is. I thought i'd try it anyway!

Thanks

#3

Status:postponed (maintainer needs more info)» postponed

The patch was written for Drupal 5.x and it wont work with Views 2 - a completely different approach is needed. The problem is that views replaces the _taxonomy_term_page page callback with its own - regardless of the relative weighting of the two modules. To get taxonomy_breadcrumb back into the loop, it will need to implement hook_menu_alter to determine if views has replaced the taxonomy_breadcrumb callback. If it has, then it will need to insert a new taxonomy_breadcrumb callback function that inserts the taxonomy breadcrumb before calling the views page callback.

I wrote a patch like this for custom breadcrumbs, and I expect the same thing would work for taxonomy_breadcrumb. But rather than duplicating code, I would suggest waiting for custom_breadcrumbs 2, which will implement taxonomy_breadcrumb as a submodule. I am working on it now, but can't promise when it will be done.

For now, I am marking this as postponed, pending the development of custom_breadcrumbs 2.

#4

Hey MGN,

Thanks for this kind explanation.

It sounds like Custom Breadcrumbs 2 will replace taxonomy breadcrumb correct? :)

Cheers,
G

nobody click here