Pagetest for translations without plural-forms information
Bug #513625 reported by
Jeroen T. Vermeulen
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Launchpad itself |
Fix Released
|
Low
|
Adi Roiban |
Bug Description
Adi Roiban notes that there are no pagetests for +translate pages in cases where no plural forms information is available.
The browser code seems to be pretty well-tested for this case (lib/lp/
In that particular case, the oopses happened on the single-message pages. A pagetest would need to include an example of those.
Related branches
lp:~adiroiban/launchpad/bug-359180-take-2
Merged
into
lp:launchpad
- Graham Binns (community): Approve (code, js)
-
Diff: 802 lines (+446/-68)11 files modifiedlib/canonical/launchpad/javascript/translations/pofile.js (+306/-2)
lib/canonical/launchpad/templates/batchnavigator-navigation-links.pt (+4/-0)
lib/lp/translations/browser/pofile.py (+20/-2)
lib/lp/translations/browser/tests/pofile-views.txt (+20/-2)
lib/lp/translations/browser/translationmessage.py (+24/-0)
lib/lp/translations/stories/standalone/xx-pofile-translate-newlines-check.txt (+2/-2)
lib/lp/translations/stories/standalone/xx-pofile-translate.txt (+33/-5)
lib/lp/translations/templates/currenttranslationmessage-translate-one.pt (+8/-11)
lib/lp/translations/templates/pofile-translate.pt (+10/-43)
lib/lp/translations/templates/translationmessage-translate.pt (+7/-0)
lib/lp/translations/templates/translations-macros.pt (+12/-1)
Changed in rosetta: | |
assignee: | nobody → Adi Roiban (adiroiban) |
tags: | added: tech-debt |
Changed in rosetta: | |
status: | New → Triaged |
importance: | Undecided → Low |
Changed in rosetta: | |
status: | Triaged → In Progress |
tags: |
added: qa-ok removed: qa-needstesting |
Changed in rosetta: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r10525 <http:// bazaar. launchpad. net/~launchpad- pqm/launchpad/ stable/ revision/ 10525>