This is very good plugin and I'd love (really need) to use it. However, the title is not translated, and it's not changing the translated strings. Could anyone please fix it? thanks.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jenlampton’s picture

Sure, which modules are you using for translation? Locale? Content Translation? i18n?
Jen

hass’s picture

Category: feature » support
Status: Needs work » Fixed

You need to configure i18n correctly. This can be done with i18n variables.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

gease’s picture

Status: Closed (fixed) » Active

Sorry, could you provide more detailed instructions of how to make taxonomy title translated? i18n settings can make anything from the variables table translatable, but this is not the case.
Any help appreciated.

jenlampton’s picture

Version: 6.x-1.0 »
Assigned: Unassigned » jenlampton

Bumping it up to the lates version so I can work on it next.

jenlampton’s picture

Status: Active » Needs review
FileSize
0 bytes

and here's a patch for i18n string support.

jenlampton’s picture

FileSize
7.35 KB

user fail on patch, trying again.

jenlampton’s picture

Status: Needs review » Fixed

fixed in 6.x-1.4 version. Marking as such.

hass’s picture

Status: Fixed » Needs work

Use module_exists() to check id i18n is installed and not function exists, please. We have this apis to use them.

jenlampton’s picture

Version: » 6.x-1.4
Status: Needs work » Closed (fixed)

function exists is faster, and module exists won't tell me if any of the functions I'm actually calling exist. :/ I don't think this is worth changing.

hass’s picture

Status: Closed (fixed) » Needs work

Use the APIs that drupal has defined. They have been made for everyone for very good reasons. Don't try to go your own way, please. Not using the API is a bug that need to be fixed.

jenlampton’s picture

Status: Needs work » Closed (fixed)

The module now works with multi-lingual, so this issue is fixed. If you have a complaint about the approach, then open an new issue and write a patch. Please do not re-open this issue.

hass’s picture

Status: Closed (fixed) » Needs work

You are not following drupal coding rules. This patch should have never ever been committed. Roll it back and do it again or fix the bug, please.

jenlampton’s picture

Status: Needs work » Closed (fixed)

Please open a new issue to correct the coding standards.