Download & Extend

page view of taxonomy with multiple-parent terms fails

Project:Taxonomy Treemenu
Version:6.x-4.2-beta1
Component:Code
Category:bug report
Priority:critical
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

Please try: http://www.ethicalemarket.com/taxmenu/menu-product-keywords-noroot2/page

Fails with:

Fatal error: Call to undefined function taxonomy_router_get_branch_title() in .../sites/all/modules/taxonomy_treemenu/taxonomy_treemenu.module on line 3308

Taxonomy terms have multiple parents and can be viewed here from Site Menu (correctly). Please note the taxonomy treemenu as it appears under the admin menus view does not reflect recent taxonomy changes performed in taxonomy manager.

In other words, the MVC is clearly broken.

Comments

#1

Hey, you are the first person to raise this issue, of which I was aware.

The handling of multiple hierarchy menus is, as you say, broken. Perhaps I would settle for this - the menus are not accurately auto-updated across multiple hierarchy taxonomies.

There is a fix. If you use the menu-rebuild feature, this always works, as far as I know.

However, this is not a particularly good fix, as one of the points of taxonomy treemenu (and similar more sophisticated modules like taxonomy VTN and taxonomy menu) is automatic upgrades. Also, a menu rebuild is inefficient on links.

The forthcoming version has new code to deal with this, which is already in place.

Leaving this issue open until the new code is released and we know the issue has been actively tested.

Rob

#2

Hey, you are the first person to raise this issue, of which I was aware.

Yes, lucky me, I have this special gift for bumping into the limits of software first, it's a lifetime blessing. Lucky lucky me, so often so lucky this way.

I will certainly test a new release on a site with a large multiple-parent taxonomy hierarchy for you (by then it might be even massive). Another lifelong blessing that I get to test these things so often first, too. Lucky me.

Webel

#3

Status:active» fixed

V5 was written partially with this in mind, and light testing has proved ok. Closing (pending more subtle bugs, I suppose)

#4

Status:fixed» closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

#5

#6

I don't find the corrected version for D6.
Still 6.4 of sept 2009.
Am I missing something?

Seems to work well anyway