don't hardcode term links, use taxonomy_term_path
joachim - August 7, 2009 - 16:24
| Project: | Taxonomy Image |
| Version: | 6.x-1.x-dev |
| Component: | Contrib Node Display |
| Category: | bug report |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | active |
Jump to:
Description
In theme_taxonomy_image_display, using taxonomy_term_path() instead of 'taxonomy/term/TID' would give other modules a chance to rewrite the path (eg forum, taxonomy_redirect).
The corresponding UI text "Link displayed Taxonomy Image to taxonomy/term/n page" should be changed at the same time.
#1
Having issues with this module and taxonomy_redirect, guessing this is the issue. I'm trying to change the output of $return_url but it seems to be having no effect.
Update:
I'm getting really frustrated with this module.
Pathauto, Taonomy_redirect, and Taxonomy Image are intercepting and rewriting my menu links from saying something like "store/(tid)" into "category/catalog/(parent_term_name)/(term_name)-0" too the entire vocabulary every time I make a change to it. Is anyone else having these issues?
#2
Pathauto makes aliases for terms by default, but if this module were behaving correctly, your redirects would kick in first.
#3
By disabling node redirect I can at least get my taxonomy views to function again, but I would prefer to keep using that module for SEO reasons. Also, I have to manually rewrite the aliases because (as we've beaten to death) Taxonomy Image disregards the default aliasing settings defined by Pathauto.
#4
It occurs to me that since this is in a theme function, you could override it to do the right thing until a fix is released.