Merge lp:~danilo/launchpad/bug-401618 into lp:launchpad
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Aaron Bentley | ||||
Approved revision: | no longer in the source branch. | ||||
Merged at revision: | 14110 | ||||
Proposed branch: | lp:~danilo/launchpad/bug-401618 | ||||
Merge into: | lp:launchpad | ||||
Diff against target: |
300 lines (+39/-55) 11 files modified
lib/lp/translations/browser/configure.zcml (+1/-1) lib/lp/translations/browser/tests/translationmessage-views.txt (+2/-4) lib/lp/translations/doc/pofile.txt (+10/-7) lib/lp/translations/doc/vpotexport.txt (+5/-3) lib/lp/translations/interfaces/potmsgset.py (+4/-6) lib/lp/translations/interfaces/translationmessage.py (+4/-0) lib/lp/translations/model/pofile.py (+1/-1) lib/lp/translations/model/potmsgset.py (+0/-3) lib/lp/translations/model/translationmessage.py (+8/-0) lib/lp/translations/templates/pofile-filter.pt (+4/-4) lib/lp/translations/tests/test_pofile.py (+0/-26) |
||||
To merge this branch: | bzr merge lp:~danilo/launchpad/bug-401618 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Aaron Bentley (community) | Approve | ||
Review via email: mp+78273@code.launchpad.net |
Commit message
[r=abentley][bug=401618] Finally remove all use of POTMsgSet.sequence in our code, to fix an odd bug or two, and allow for column removal from the database.
Description of the change
= Bug 401618 =
This is a bug about remaining use of POTMsgSet.sequence that stopped us from killing this column in the database: nature of POTMsgSet has changed in the meantime so it can participate in multiple different translation templates, and sequence in each of those templates is stored on TranslationTemp
Biggest issue was the fact that we constructed URLs for TranslationMessages using potmsgset.sequence. To aid with that, we instead introduce TranslationMess
As for all the other uses, I replace potmsgset.sequence with potmsgset.
The tests previously used to pass because they didn't heavily depend on this change (eg. the fact that potmsgset.
== Tests ==
"bin/test -vvm lp.translations" still passes
== Demo and Q/A ==
Browse around +translate and +filter pages on translations.l.n, especially clicking on the "zoom" icon for each of the messages. Also look for translation suggestions saying "used in" and "suggested in" and follow those links to ensure they point to right pages.
= Launchpad lint =
NOTE: I'll fix lint after the review, to simplify the review.
Checking for conflicts and issues in changed files.
Linting changed files:
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
lib/lp/
./lib/lp/
1: narrative uses a moin header.
18: narrative uses a moin header.
./lib/lp/
196: Line exceeds 78 characters.
198: Line exceeds 78 characters.
343: E251 no spaces around keyword / parameter equals
./lib/lp/
19: 'IntCol' imported but unused
./lib/lp/
32: Line has trailing whitespace.
Looks good to me.