Closed (fixed)
Project:
IE 6 Update
Version:
6.x-1.4
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
15 May 2012 at 11:12 UTC
Updated:
18 Aug 2012 at 22:01 UTC
In the function ie6update_footer() in ie6update.module, we see
$message = variable_get('ie6update_update_bar_message', 'Internet Explorer is missing updates required to view this site. Click here to update... ');
followed by
t(filter_xss_admin($message))
This post explains in detail why this is considered a very bad practice that should be changed.
I'm sorry I don't have time to propose a patch at this time; I'm not yet familiar enough with the best alternative solutions to just conjure one up, but the way towards a partial solution is described here, with a link to a more thorough approach.
Comments
Comment #1
iamEAP commentedThanks for the report, martin_q. I'm guessing this is a typo leftover from the last time this was brought up in this module (#951530: Document how to translate the link and text).
If you're willing, you could patch and test to make sure translations still come through. The proper way to translate the variables should already be documented in the README.txt.
Comment #2
iamEAP commentedFixed.