Closed (fixed)
Project:
Term Display
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
5 Apr 2009 at 13:19 UTC
Updated:
5 Dec 2009 at 14:50 UTC
Jump to comment: Most recent file
Comments
Comment #1
joachim commentedWhat would be needed to make this work?
Are there API docs for localized terms? Which module is this, anyway?
Comment #2
michaelcrm commentedI totally agree that this module need to be compatable with Localized Terms! What should I do to make this work?
Any ideas?
Comment #3
nedjoComment #4
michaelcrm commentedAnyone has tested the #3 patch yet?
Comment #5
nedjoI was sure I'd put a comment along with the patch. Oh well, here it is again:
Untested patch attached, please test and report back. Thanks!
Comment #6
nedjo@michaelcrm, binhcan:
I'd appreciate it if you would follow up on your feature request by testing the patch I produced.
See http://drupal.org/node/60108 for instructions on applying patches. If it's not something you've done before, it really doesn't take a lot of time to learn.
Thanks.
Comment #7
smk-ka commentedI've had success using an alternative approach, by just bumping the module weight of term_display to run after i18ntaxonomy, but before views (i.e., the very same technique that i18ntaxonomy itself uses to localize the node terms). This frees us from adding explicit support for i18ntaxonomy.
Comment #8
sunNot really sure which approach is better, but it seems we have two working approaches now. :)
Comment #9
gmreed commentedI have just set up Term Display using Nedjo's patch (#3 above) and it is working flawlessly so far - thanks Nedjo! I will report back if I run into any issues.
Comment #10
anasynth commentedI know its a little late but I've just tried out the patch in #3 and just thought I'd add that it seems to work fine. Thanks for the great module.
Comment #11
smk-ka commentedAfter comparing Nedjo's patch with i18ntaxonomy I've decided it is definitely a more solid approach than mine. Committed #3 with a tiny change to prevent a PHP notice if i18ntaxonomy isn't installed. Thanks all!