Launchpad itself

Merge lp://qastaging/~wgrant/launchpad/bug-736005-begins into lp://qastaging/launchpad

  1. bug-736005-begins
  2. Merge into devel
Proposed by William Grant
Status: Merged
Merged at revision: 17046
Proposed branch: lp://qastaging/~wgrant/launchpad/bug-736005-begins
Merge into: lp://qastaging/launchpad
Diff against target: 104 lines (+9/-37)
4 files modified
lib/lp/translations/browser/tests/translationmessage-views.txt (+0/-27)
lib/lp/translations/browser/translationmessage.py (+2/-7)
lib/lp/translations/model/translationmessage.py (+6/-2)
lib/lp/translations/templates/currenttranslationmessage-translate-one.pt (+1/-1)
To merge this branch: bzr merge lp://qastaging/~wgrant/launchpad/bug-736005-begins
Reviewer Review Type Date Requested Status
Celso Providelo (community) Approve
Review via email: mp+223035@code.qastaging.launchpad.net

Commit message

Turn TranslationMessageMixIn.sequence into a cachedproperty, invalidating it when browser_pofile is set. Use it everywhere possible, cutting POFile:+translate query counts by up to 20%.

Description of the change

Turn TranslationMessageMixIn.sequence into a cachedproperty, invalidating it when browser_pofile is set. Use it everywhere possible, cutting POFile:+translate query counts by up to 20%.

To post a comment you must log in.
Revision history for this message
Celso Providelo (cprov) wrote :

Let's crop this low-hanging fruit, very good!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.