Multilingual Support for Term Display, please

binhcan - April 5, 2009 - 13:19
Project:Term Display
Version:6.x-1.x-dev
Component:Code
Category:feature request
Priority:normal
Assigned:Unassigned
Status:fixed
Description

I love this module. Please make it compatable with Localized Terms. Thanks.

At the moment it only show the original strings, not the translated one. :((

#1

joachim - April 6, 2009 - 19:58
Status:active» postponed (maintainer needs more info)

What would be needed to make this work?
Are there API docs for localized terms? Which module is this, anyway?

#2

michaelcrm - May 5, 2009 - 09:08

I totally agree that this module need to be compatable with Localized Terms! What should I do to make this work?

Any ideas?

#3

nedjo - May 5, 2009 - 12:42
Status:postponed (maintainer needs more info)» needs review
AttachmentSize
localize-term-names-424326-3.patch 1.69 KB

#4

michaelcrm - May 6, 2009 - 02:15

Anyone has tested the #3 patch yet?

#5

nedjo - May 6, 2009 - 12:34

I 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!

#6

nedjo - May 8, 2009 - 20:06

@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.

#7

smk-ka - August 21, 2009 - 18:18

I'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.

AttachmentSize
term_display_i18ntaxonomy.patch 529 bytes

#8

sun - August 21, 2009 - 19:44
Status:needs review» reviewed & tested by the community

Not really sure which approach is better, but it seems we have two working approaches now. :)

#9

gmreed - November 1, 2009 - 19:36

I 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.

#10

anasynth - November 20, 2009 - 13:37

I 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.

#11

smk-ka - November 21, 2009 - 14:41
Status:reviewed & tested by the community» fixed

After 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!

 
 

Drupal is a registered trademark of Dries Buytaert.