Doing a multi-lingual site and it'd be great to have the node's language as a token to pass to pathauto.

CommentFileSizeAuthor
#2 token_233076_2.patch1.3 KBgreggles
#1 token_233076.patch1.24 KBdrewish
token_language.diff1.17 KBdrewish
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drewish’s picture

drewish CreditAttribution: drewish commented
FileSize
token_233076.patch1.24 KB

here's a re-roll.

i should add that this is now a core field so it doesn't belong in the i18n per #176449: Token support (primarily for pathauto).

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
FileSize
token_233076_2.patch1.3 KB

Added filtering and a little if statement so it gets set to blank string in case there is no language (which I assume can happen...).

drewish’s picture

drewish CreditAttribution: drewish commented

greggles, i guess the xss filtering can't hurt but it does see a little over the top, core doesn't run the role names through a filter but like i said it doesn't hurt anything.

the language should always be set--even if it's just to an empty string--since it's loaded from the node table.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Needs review » Fixed

Great - I removed the crazy if, left in the filtering, and applied this to the 6.x branch.

Anonymous’s picture

Anonymous (not verified) CreditAttribution: Anonymous commented
Status: Fixed » Closed (fixed)

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

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan commented

For contexts outside of nodes, see #975116: Create a 'language' token type (D7) (for content and interface languages).