not that it was of utter importance, but when i went to f.e. http://hfopi.org/de/sendmail/projektablaufplan and saw the links for the different languages broken, i took the time to repair. Patch has been thoroughly tested and approved for wildlife ;)

CommentFileSizeAuthor
print-links-translation.patch3.16 KBeMPee584

Comments

eMPee584’s picture

eMPee584 commented
Version: 7.x-1.x-dev » 6.x-1.x-dev

oops thats D6 here..

jcnventura’s picture

jcnventura
Primary language English
commented

I don't understand why this is needed... Can this be a duplicate of #372387: possibility to re-import all .po files?

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Needs review » Postponed (maintainer needs more info)

I don't understand why this is needed... Can this be a duplicate of #372387: possibility to re-import all .po files?

eMPee584’s picture

eMPee584 commented

lols nooo ;)
it's really only a minor issue, but without this code broken links are displayed in the locale block. If there is an English language node with the alias foobar, the links in the locale block on the page /en/printmail/foobar point to /[langcode]/printmail/foobar. This fails in the case that there is f.e. a German version of the same node with alias fuhbar. Without this code, the German language link on that path will point to /de/printmail/foobar instead of /de/printmail/fuhbar and might produce a 404 depending on multilingual settings.
So this hook implementation basically applies the work of translation_translation_link_alter() to links with print/printmail/printpdf prefix. Any clarities remaining?

jcnventura’s picture

jcnventura
Primary language English
commented
Status: Postponed (maintainer needs more info) » Needs review
jcnventura’s picture

jcnventura
Primary language English
commented
Issue summary: View changes
Status: Needs review » Closed (outdated)

Closing all issues related to the Drupal 6 version of the print module.