Not sure if there's any side effects, but I was having issues with a view in Workbench whereby the path alias wasn't being created for nodes in a different language; I eventually tracked this down to Domain Path and a call to drupal_get_path_alias which didn't pass in the language variable.

CommentFileSizeAuthor
#3 1412040-path-lauguage_0.patch2.24 KBagentrickard
#2 1412040-path-lauguage.patch1.23 KBagentrickard
#1 1412040-path-lauguage.patch1.18 KBagentrickard
domain_path_language.patch414 bytesapemantus
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
FileSize
1412040-path-lauguage.patch1.18 KB

Makes sense. Note that because of the IF/ELSE structure, the original patch did not ensure that $language was set properly.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
FileSize
1412040-path-lauguage.patch1.23 KB

Clarifies the comment line about language handling.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard commented
Status: Needs review » Fixed
FileSize
1412040-path-lauguage_0.patch2.24 KB

Cleaned up a bit and committed.

5b5e0c8..709d4b6 7.x-1.x -> 7.x-1.x

Status: Fixed » Closed (fixed)

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