The links to the the terms are manually build.
In function _taxonomy_list_get_table there is:

<a href="'. base_path() .'taxonomy/term/'. $term->tid .'">'. taxonomy_image_display($term->tid, '') .'</a>

This only works if you have search-friendly-urls enabled.
Maybe l() or url() should be used here.

Thanks for this wonderful module
Stefan

CommentFileSizeAuthor
#1 taxonomy_list_links.patch1.15 KBmoonray

Comments

moonray’s picture

moonray commented
Version: 5.x-1.x-dev » master
Assigned: Unassigned » moonray
Status: Active » Needs review
StatusFileSize
new taxonomy_list_links.patch1.15 KB

Patch to fix the links.

BoogieBug’s picture

BoogieBug commented
Assigned: moonray » BoogieBug
Status: Needs review » Fixed

Patch was applied. Not yet tested with taxonomy_image.module.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
deavidsedice’s picture

deavidsedice commented

This patch is not working with images. I've downloaded today the stable version, and I've seen this:

<img src='http://iluminacion-mv.com/files/category_pictures/aplique.jpg' width='128' height='128' alt='Apliques' />
Apliques
  <img src='http://iluminacion-mv.com/files/category_pictures/colgantes-regletas.jpg' width='128' height='128' alt='Colgantes regletas' />
Colgantes regletas
Colgantes SB

It prints the code, not the resulting HTML.

Why?

Because you're using the l() function without specify that's HTML in the text.
See the API for l():
http://api.drupal.org/api/5/function/l

Note this:
$html Whether the title is HTML, or just plain-text. For example for making an image a link, this must be set to TRUE, or else you will see the encoded HTML.

I corrected this for my site, changing the code in the line 266 with:

  .  l(taxonomy_image_display($term->tid, ''), 'taxonomy/term/'. $term->tid, array(), NULL, NULL, FALSE, TRUE)

If you want a patch, make me know. (I think this will be enough to repair the bug)

Tom-182’s picture

Tom-182 commented

@ deavidsedice
Thank you so much, your code works perfectly.