BadPluralExpression: Number of recognized plural forms doesn't match the expected number of them.
Bug #537100 reported by
Jeroen T. Vermeulen
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Jeroen T. Vermeulen | ||
Bug Description
We just had an Oops in friendly_
BadPluralExpr
OOPS-1530EB1048
Related branches
lp:~jtv/launchpad/bug-537100
- Henning Eggers (community): Approve (code)
-
Diff: 275 lines (+99/-52)4 files modifiedlib/lp/translations/browser/language.py (+21/-7)
lib/lp/translations/browser/tests/language-views.txt (+14/-0)
lib/lp/translations/utilities/pluralforms.py (+41/-33)
lib/lp/translations/utilities/tests/test_pluralforms.py (+23/-12)
| description: | updated |
Revision history for this message
| Jeroen T. Vermeulen (jtv) wrote | #1 |
| Changed in rosetta: | |
| assignee: | nobody → Jeroen T. Vermeulen (jtv) |
| status: | Triaged → In Progress |
| milestone: | none → 10.04 |
Revision history for this message
| Ursula Junque (ursinha) wrote Bug fixed by a commit | #2 |
| Changed in rosetta: | |
| status: | In Progress → Fix Committed |
| tags: | added: qa-needstesting |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in rosetta: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Turns out the plural expressions we had for Welsh and Lower Sorbian really were wrong. The Welsh one had a "||" where it should have had "&&"; Lower Sorbian had a dedicated form for 0 that wasn't reflected in the nplurals. I've fixed these in the database. (As far as I could tell there are no translation messages in Lower Sorbian that use plurals so far).
It would also be good to have this validated when submitting language admin pages.