Opened 11 years ago
Closed 11 years ago
#20441 closed Bug (invalid)
Swedish ordinal numbering in django.contrib.humanize is wrong
Reported by: | Horst Gutmann | Owned by: | Horst Gutmann |
---|---|---|---|
Component: | Translations | Version: | dev |
Severity: | Normal | Keywords: | |
Cc: | Triage Stage: | Ready for checkin | |
Has patch: | yes | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
It seems like the current translation strings for ordinals the Swedish .po files are wrong. 2 should for instance become 2:a but ends up being 2:e.
The underlying problem (outside of the messages themselves) should be beyond the scope of this ticket and will probably also go beyond #15156, but at least for the first couple of numbers this should be corrected.
Change History (6)
comment:1 by , 11 years ago
comment:2 by , 11 years ago
Owner: | changed from | to
---|---|
Status: | new → assigned |
comment:3 by , 11 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:4 by , 11 years ago
Pull request: https://github.com/django/django/pull/1118
I've included the translation directly into the repository and will notify the i18n team ASAP about this change in order to prevent overriding the changes with the next Transifex update.
comment:5 by , 11 years ago
Has patch: | set |
---|---|
Triage Stage: | Accepted → Ready for checkin |
It'd be nice to have this fixed in Transifex before closing the ticket, so I'll keep the PR on hold for now. Otherwise it's fine.
comment:6 by , 11 years ago
Resolution: | → invalid |
---|---|
Status: | assigned → closed |
Thanks for the report and the fix, but pure translation bugs are not tracked by tickets (no code change). If we add a test for Swedish, why wouldn't we add tests for the tenth of other languages?
So please, fix it on Transifex directly and this will be included in the next Django release.
Test-case do demonstrate the issue:
https://github.com/zerok/django/commit/a3ebb50e8b1c67a6312b89103376a7d82ce9ea17