#20441 closed Bug (invalid)

Swedish ordinal numbering in django.contrib.humanize is wrong

Reported by: zerok Owned by: zerok
Component: Translations Version: master
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 Changed 22 months ago by zerok

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset

comment:2 Changed 22 months ago by zerok

  • Owner changed from nobody to zerok
  • Status changed from new to assigned

comment:3 Changed 22 months ago by EmilStenstrom

  • Triage Stage changed from Unreviewed to Accepted

comment:4 Changed 22 months ago by zerok

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 Changed 22 months ago by aaugustin

  • Has patch set
  • Triage Stage changed from Accepted to 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 Changed 22 months ago by claudep

  • Resolution set to invalid
  • Status changed from assigned to 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.

Note: See TracTickets for help on using tickets.
Back to Top