Closed
Bug 797984
Opened 12 years ago
Closed 3 years ago
The *_lang_files tags for setting lang files in a template don't work quite right
Categories
(www.mozilla.org :: Bedrock, defect, P4)
www.mozilla.org
Bedrock
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: pmac, Assigned: pmac)
References
(Blocks 1 open bug)
Details
(Whiteboard: u=developer c=l10n p=3)
In doing some testing on Bug #783202 I found that the `{% add_lang_files %}` and `{% set_lang_files %}` tags don't work as advertised. The lang files set at the end are supposed to contain the ones for the rendered file, any ones in parent templates, and the defaults, in that order. The inheritance is not working at all.
Also they only work from within blocks, but most of the use of these tags in our templates is from outside of blocks.
This requires more testing and possibly a rework of how these are built.
Some initial tests have been added in `lib/l10n_utils/tests/test_template.py`.
Updated•12 years ago
|
Component: L10N → Bedrock
Updated•12 years ago
|
Blocks: bedrock-l10n
Comment 1•12 years ago
|
||
Does this impact the l10n work needed for MWC?
Assignee | ||
Comment 2•12 years ago
|
||
I don't believe so. We've been fine so far with the way it is and I believe we can continue on for the foreseeable future. It's just not the way it "should" be, but it's not a high priority.
Updated•12 years ago
|
Whiteboard: u=developer c=l10n p=3 → u=developer c=l10n p=4
Assignee | ||
Comment 3•12 years ago
|
||
I think :jbertsch meant to make it a P4 priority.
Priority: P2 → P4
Whiteboard: u=developer c=l10n p=4 → u=developer c=l10n p=3
Assignee | ||
Comment 4•3 years ago
|
||
Fixed because we've removed lang file support from the site.
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•