Problem/Motivation

i18n_hreflang module provides its own implementation of the said hook, but only for the front page.

Proposed resolution

Simply reuse the code from the i18n_hreflang module and add an if clause to limit the translation to node paths only (the ones the code it most likely to work correctly). url() function seems to be handling these without any issues.

Remaining tasks

Further review & testing is required to determine whether it work properly for all node paths (the code has so far been tested on both aliased and unaliased paths on a local dev site).

The code could also be expanded to enable entity path translation.

Files: 
CommentFileSizeAuthor
#6 1970812_support_hook_i18n_translate_path.patch937 bytesmaximilianmikus
#5 1970812_support_hook_i18n_translate_path.patch937 bytesmaximilianmikus
#2 0001-Add-hook_i18n_translate_path-for-Entity-Translation.patch1.54 KBisakic
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 0001-Remaking-patch.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]
0001-Add-hook_i18n_translate_path-for-Entity-Translation-.patch1.54 KBisakic
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 0001-Add-hook_i18n_translate_path-for-Entity-Translation-.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Comments

Status:Needs review» Needs work

The last submitted patch, 0001-Add-hook_i18n_translate_path-for-Entity-Translation-.patch, failed testing.

StatusFileSize
new1.54 KB
FAILED: [[SimpleTest]]: [MySQL] Unable to apply patch 0001-Remaking-patch.patch. Unable to apply patch. See the log in the details link for more information.
[ View ]

Status:Needs work» Needs review

Status:Needs review» Needs work

The last submitted patch, 0001-Remaking-patch.patch, failed testing.

Fixing the patch.

argh..! had the branches switched up...

Category:task» feature

Not sure how to implement that properly but right now we are supporting only nodes. Moreover, if I understand correctly what the code is trying to achieve, it seems we are not taking into account whether translations are actually available.