In "function theme_content_taxonomy_formatter_link($element) {" the link url is set using "taxonomy_term_path($term)" which is good, except that some modules will want to alter this path (e.g. http://drupal.org/project/nat) and your method of getting the URL doesn't take this into account.

Instead, it should simply link to "'taxonomy/term/'. $term->tid" (as the core taxonomy module does) and let drupal worry about where that needs to go.

Patch attached

CommentFileSizeAuthor
content_taxonomy_links.patch1.84 KBa_c_m

Comments

mh86’s picture

mh86 commented

Hi!

As far as I can see, the core taxonomy module uses taxonomy_term_path too (e.g. in taxonomy_link). And if we do not use taxonomy_term_path we drop support for other modules overwriting the path of terms managed by themselves. Or does this still work?

a_c_m’s picture

a_c_m commented

humm... your right.

Not sure how to get around this - but i know the current system does not work with NAT as its expecting to overrule 'taxonomy/term/'. $term->tid not something else. Perhaps the issue is with NAT then?

magnus’s picture

magnus commented
Status: Active » Closed (works as designed)

Reopen issue if problem still exists.