Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#9634 closed (fixed)

Missing plural string in Slovak translation

Reported by: claude@… Owned by: nobody
Component: Translations Version: dev
Severity: Keywords:
Cc: Marian Andre, marcel Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

The Slovak django.po is missing a Plural string. Even when Django "sub-apps" provide a correct Slovak po file with a Plural string, Django uses the default one (Germanic-based). I suggest to never accept a po file in Django without a Plural string in the header.
The Plural string for Slovak is:
"Plural-Forms: nplurals=3; plural= (n==1) ? 1 : (n>=2 && n<=4) ? 2 : 0;"

Attachments (3)

djangojs.po (3.1 KB ) - added by marcel 16 years ago.
Slovak djangojs.po file
django.po (104.5 KB ) - added by marcel 16 years ago.
Slovak django.po file
django-sk.diff (55.0 KB ) - added by Pavol 'Scrool' Babinčák 16 years ago.
Diff of .po files by marcel against files in svn repository. I haven't done any changes. Just a note that it contains much more fixes not only reported bug.

Download all attachments as: .zip

Change History (13)

comment:1 by Malcolm Tredinnick, 16 years ago

Cc: Marian Andre added

Please create a patch to the django.po file (djangojs.po has no plural forms) to add this change and also update the current plural msgstrs to have the right 'n' values for the translations. The line you're proposing to add makes the existing msgstrs for plural forms incorrect (they current only have 0 and 1), so I don't want to just apply it without fixing those. You don't have to completely update the PO file or anything: just fix the indices on any msgids that have plural forms.

Adding mandre to the CC list, since he did the last Slovak translation and might also be able to help here (Marian, please feel free to remove yourself if you're not interested in following this).

comment:2 by claude@…, 16 years ago

I'm completely illiterate in Slovak. It'd be much better if mandre could fix it.

comment:3 by Claude Paroz, 16 years ago

Moreover the date format strings are wrongly translated:

#: utils/translation/trans_real.py:399
msgid "DATE_FORMAT"
msgstr "DÁTUM_FORMÁT"

#: utils/translation/trans_real.py:400
msgid "DATETIME_FORMAT"
msgstr "DÁTUMAČAS_FORMÁT"

#: utils/translation/trans_real.py:401
msgid "TIME_FORMAT"
msgstr "ČAS_FORMÁT"

#: utils/translation/trans_real.py:417
msgid "YEAR_MONTH_FORMAT"
msgstr "ROK_MESIAC_FORMÁT"

#: utils/translation/trans_real.py:418
msgid "MONTH_DAY_FORMAT"
msgstr "MESIAC_DEŇ_FORMÁT"

This triggers server errors mentionned in ticket:10048

comment:4 by marcel, 16 years ago

If there is nobody else who could update the Slovak translation po file to have this bug fixed I could do it (as a native Slovak speaker). Please send me (or point me to) current Slovak po file and I'll try to do my best.

Thanks.

comment:5 by marcel, 16 years ago

marcel (at) telka (dot) sk

comment:6 by Claude Paroz, 16 years ago

The current Slovak po files are available here:
source:django/trunk/django/conf/locale/sk/LC_MESSAGES

by marcel, 16 years ago

Attachment: djangojs.po added

Slovak djangojs.po file

by marcel, 16 years ago

Attachment: django.po added

Slovak django.po file

comment:7 by marcel, 16 years ago

Cc: marcel added
Has patch: set

Please integrate attached djangojs.po and django.po files. All known problems are fixed there.

Thanks.

in reply to:  7 comment:8 by Marc Fargas, 16 years ago

Triage Stage: UnreviewedAccepted

Replying to marcel:

Please integrate attached djangojs.po and django.po files. All known problems are fixed there.

*Please* as stated in the first comment and in the documentation. Attach a patch (diff) file; It makes things much easier for us. There are lots of ways for doing so. If you are running SVN: "svn diff > mypatch.diff"

Thanks!!

by Pavol 'Scrool' Babinčák, 16 years ago

Attachment: django-sk.diff added

Diff of .po files by marcel against files in svn repository. I haven't done any changes. Just a note that it contains much more fixes not only reported bug.

comment:9 by Malcolm Tredinnick, 16 years ago

Resolution: fixed
Status: newclosed

(In [9913]) Fixed #9634 -- Updated Slovak translation. Thanks Marcel Telka and scrool.

comment:10 by Malcolm Tredinnick, 16 years ago

(In [9914]) [1.0.X] Fixed #9634 -- Updated Slovak translation. Thanks Marcel Telka, Marian Andre and scrool.

Backport of r9913 and r9263 from trunk.

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