Hi Jose,

I would like to ask for your attentention to an issue in the pathauto module queue.

As mentioned here http://drupal.org/node/290421#comment-3807650 it would be nice to have i18n version tokens of catpath or termpath available for pathauto module. That way localized terms can be used in paths generated by the pathauto module.

The full issue can be found here; http://drupal.org/node/290421

Do you think this is a realistic feature request?

Cheers,
Bartezz

CommentFileSizeAuthor
#7 1014318-support-localized-taxonomy-DRUPAL-6--1.patch4.8 KBpokurek
#1 1014318-support-localized-taxonomy-DRUPAL-6--1.patch4.48 KBpokurek

Comments

pokurek’s picture

pokurek commented
Version: 6.x-1.7 » 6.x-1.x-dev
StatusFileSize
new 1014318-support-localized-taxonomy-DRUPAL-6--1.patch4.48 KB

Hi,

As mentioned before, it could be nice to have for pathauto module something like this patch.
Patch is not so well tested yet, but I hope it is good for start think about.

Have a nice time,
Petr

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented

subscribing

mxt’s picture

mxt
Primary language Italian
Location Milan
commented

Subscribing

philipz’s picture

philipz commented

subscribing

anrikun’s picture

anrikun commented

@pokurek: patch at #1 doesn't work for me...
Is this supposed to work with "Localize terms. Terms are common for all languages, but their name and description may be localized."?

bartvdputte’s picture

bartvdputte commented

This doesn't seem to work for me neither with "localize terms"...

pokurek’s picture

pokurek commented
StatusFileSize
new 1014318-support-localized-taxonomy-DRUPAL-6--1.patch4.8 KB

I have rewritten the patch. Try this one.

If term is from vocabulary with "Localize terms." then it uses i18nstrings() function for term name translation.

nimek’s picture

nimek commented

#7 Not working for localize terms vocabulary settings cause in that case $term->language is empty.

mathieu’s picture

mathieu commented

I have a feeling this could include (or be included in) #1141246: Add new tokens (i18n-termpath-raw and i18n-termpath).

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented
Status: Active » Needs review

...this is NR based on #7.

pokurek’s picture

pokurek commented

#8 I think, If we don't know term language, we can't do any translations.
So, there is a condition for that and if term language is empty, the function return term itself.

if (i18ntaxonomy_vocabulary($term->vid) == I18N_TAXONOMY_LOCALIZE && $term->language) {
  // do translation
}
else {
  $values['i18n-term-raw'] = $term->name;
  $values['i18n-term'] = check_plain($term->name);
  // and so on
}

Module pathauto set term language before it is translated as a path alias (can see in issue: http://drupal.org/node/290421).

jose reyero’s picture

jose reyero commented
Status: Needs review » Closed (won't fix)

No new features for 6.x

klonos’s picture

klonos
he/him
Primary language English
Location 90% Melbourne, Australia - 10% Larissa, Greece
commented
Version: 6.x-1.x-dev » 7.x-1.x-dev
Category: feature » support
Status: Closed (won't fix) » Active

...sorry for reopening this, but this time it is a support question: How is this fixed in 7.x?

jose reyero’s picture

jose reyero commented
Status: Active » Closed (duplicate)

See #1177084: Translated vocabulary names not working with Pathauto