Closed (fixed)
Project:
Forward
Version:
6.x-1.19
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
20 Jan 2011 at 10:27 UTC
Updated:
10 Aug 2011 at 00:42 UTC
Jump to comment: Most recent file
Hi,
I found out that the title of the page where the user is referred to by clicking on the "Send by Emal" link is not translated - and I could not find the place where to add the "t()" function in the module code.
Best,
Tobias
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | page-title-not-translated-1033660-4.patch | 1.86 KB | zilverdistel |
Comments
Comment #1
pandersb commentedYes, I have the same problem.
The title of the page uses drupal_set_title in the function forward_page(). So even when I tried hook_menu_alter, the title would still get overridden in line 491 of forward.module.
I needed to wrap the variable with the t function. This is not ideal since it'll conflict when I do updates of this module.
drupal_set_title(check_plain(t(variable_get('forward_page_title', 'Forward this page'))));Comment #2
tobiberlinCould anyone help please?
Comment #3
tobiberlinagain: any suggestions?
Comment #4
zilverdistel commentedThis is indeed an annoying bug.
On line 62:
I don't see why the page callback should use
drupal_set_title()(on line 490) to reset the page title. Titles defined in hook_menu are translated automatically, drupal_set_title() doesn't take care of that.Also, I noticed the variable 'forward_page_title' never gets set. This is the output of
grep -rni forward_page_title *My guess is that this should be changed to
variable_get('forward_email_title', t('Forward this page'))in these cases. The included patch fixes it for me (applies to git-version).Comment #5
john.oltman commentedPatch worked for me, just committed to 6.x branch. I also picked up the epostcard title while I was in there. I will create a new production release after resolving an issue with adding Mollom support.
Comment #6
zilverdistel commentedThanx!
Comment #7
john.oltman commented