Last Comment Bug 746564 - Replace 'request.locale' with 'request.LANGUAGE_CODE'
: Replace 'request.locale' with 'request.LANGUAGE_CODE'
u=dev c=general p=0 s=2013.3 [qa-]
Classification: Other
Component: General (show other bugs)
: unspecified
: All All
-- normal (vote)
: 2013Q1
Assigned To: Michał Frontczak [:fxa90id] ♥
Depends on:
  Show dependency treegraph
Reported: 2012-04-18 07:32 PDT by James Socol [:jsocol, :james]
Modified: 2013-02-07 17:31 PST (History)
4 users (show)
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Description User image James Socol [:jsocol, :james] 2012-04-18 07:32:02 PDT
The Django docs[1] indicate that the built-in translation middleware (which we don't use) prefers 'request.LANGUAGE_CODE'. For compatibility with 3rd party apps (and to make a waffle upgrade possible) we should switch.

This if probably a big patch, but almost entirely search-and-replace, unless some lines need to be rewrapped.

Comment 1 User image Will Kahn-Greene [:willkg] ET needinfo? me 2012-04-18 11:04:45 PDT
To clarify, is this something we should fix sooner rather than later, or is this something we can stick in the code quality pile to be fixed in an upcoming house-keeping week?
Comment 2 User image Apokalyptica79 2012-09-11 07:22:55 PDT
Hello i would like to try this.

Comment 3 User image Ricky Rosario [:rrosario, :r1cky] 2012-09-11 07:26:35 PDT
Let me know if you have any questions!
Comment 4 User image Michał Frontczak [:fxa90id] ♥ 2013-02-05 16:48:00 PST
Can I pick it?
Comment 5 User image Ricky Rosario [:rrosario, :r1cky] 2013-02-05 17:00:24 PST
(In reply to Michał Frontczak :fxa90id from comment #4)
> Can I pick it?

go for it :)
Comment 6 User image Michał Frontczak [:fxa90id] ♥ 2013-02-05 17:14:56 PST
Comment 7 User image Ricky Rosario [:rrosario, :r1cky] 2013-02-07 16:55:12 PST
Landed on master:
Comment 8 User image Ricky Rosario [:rrosario, :r1cky] 2013-02-07 17:31:16 PST
Deployed to prod.
Comment 9 User image Ricky Rosario [:rrosario, :r1cky] 2013-02-07 17:31:32 PST
Thanks Michał this was awesome!

Note You need to log in before you can comment on or make changes to this bug.