Opened 7 years ago

Last modified 2 years ago

#14129 new Bug

Slovenian translation plural-forms

Reported by: Gasper Zejn Owned by: Jannis Leidel
Component: Internationalization Version: 1.2
Severity: Normal Keywords:
Cc: diegobz@… Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no


Adds translation for [13502] and also fixes the wrong plurals equation.

Attachments (2)

django-i18n-sl-1.2.x.patch (32.3 KB) - added by Gasper Zejn 7 years ago.
Updated diff of django.po
14129.diff (10.6 KB) - added by Claude Paroz 6 years ago.
Only plural forms changed

Download all attachments as: .zip

Change History (21)

Changed 7 years ago by Gasper Zejn

Attachment: django-i18n-sl-1.2.x.patch added

Updated diff of django.po

comment:1 Changed 6 years ago by Russell Keith-Magee

Triage Stage: UnreviewedReady for checkin

comment:2 Changed 6 years ago by Claude Paroz

Summary: Updated Slovenian translation for 1.2.xFix Slovenian translation plural-forms
Triage Stage: Ready for checkinAccepted

Django translations are now done through the Transifex project.

You will find all required information on the Django documentation website:

I'm just letting the bug open for the plural-form change. We need to check if it is feasible through Transifex.

comment:3 Changed 6 years ago by Gasper Zejn

Resolution: invalid
Status: newclosed

Uh, seems I made a mistake and haven't double checked the plural equation after a bug report from a user.

Please discard this patch as the current plural equation is valid. I'm closing this bug as invalid.


comment:4 Changed 6 years ago by Claude Paroz

Yes, the current equation is valid. It's just that more and more projects are using another one. Read the rationale here:
See also:

But as sl coordinator, you are the one to decide :-)

comment:5 Changed 6 years ago by Gasper Zejn

Resolution: invalid
Status: closedreopened

After a discussion with fellow translators, we've decided we should adopt the new equation.

Can we do this before 1.3 release?

comment:6 Changed 6 years ago by Jannis Leidel

milestone: 1.3

Yeah, we can do this, is the attached patch the one to commit?

Changed 6 years ago by Claude Paroz

Attachment: 14129.diff added

Only plural forms changed

comment:7 Changed 6 years ago by Jannis Leidel

Owner: changed from nobody to Jannis Leidel
Status: reopenednew

comment:8 Changed 6 years ago by Jannis Leidel

FYI, we can't really fix this bug in Django since Transifex handles plural forms for us. I've filed a bug report there to make sure we don't forget this.

Last edited 6 years ago by Jannis Leidel (previous) (diff)

comment:9 Changed 6 years ago by Jannis Leidel

Okay, the Transifex developers now have tested it on and ask us if the following plural form example is correct:

# This file is distributed under the same license as the Django package.
msgid ""
msgstr ""
"Project-Id-Version: Django\n"
"POT-Creation-Date: 2010-05-13 15:35+0200\n"
"PO-Revision-Date: 2011-02-14 16:14+0000\n"
"Last-Translator: Django team\n"
"Language-Team: English <>\n"
"MIME-Version: 1.0\n"
"Content-Type: text/plain; charset=UTF-8\n"
"Content-Transfer-Encoding: 8bit\n"
"Language: sl\n"
"Plural-Forms: nplurals=4; plural=(n%100==1 ? 1 : n%100==2 ? 2 : n%100==3 || n%100==4 ? 3 : 0)\n"

#: contrib/admin/templates/admin/change_form.html:39
#: contrib/admin/templates/admin/change_list.html:71
#: contrib/admin/templates/admin/auth/user/change_password.html:24
#: contrib/admin/templates/registration/password_change_form.html:15
msgid "Please correct the error below."
msgid_plural "Please correct the errors below."
msgstr[0] "Prosimo, odpravite sledečo napako."
msgstr[1] "Prosimo, odpravite sledeči napaki."
msgstr[2] "Prosimo, odpravite sledeče napake."
msgstr[3] "Prosimo, odpravite sledeče napake."

comment:10 Changed 6 years ago by diegobz@…

Cc: diegobz@… added

comment:11 Changed 6 years ago by Gasper Zejn

No, this isn't correct. The plural forms need shuffling around using this pseudocode assignment:

msgstr[0] = msgstr[3]
msgstr[1] = msgstr[0]
msgstr[2] = msgstr[1]
msgstr[3] = msgstr[2]


comment:12 Changed 6 years ago by Jannis Leidel

For the record, the Transifex developers are working on an update but can't give a timeline since changing the order of the plural forms isn't supported at the moment. I doubt this will be fixed in time before the final 1.3 release.

Last edited 6 years ago by Jannis Leidel (previous) (diff)

comment:13 Changed 6 years ago by patchhammer

Easy pickings: unset
Patch needs improvement: set
Severity: Normal
Type: Uncategorized

14129.diff fails to apply cleanly on to trunk

comment:14 Changed 6 years ago by Julien Phalip

Type: UncategorizedBug

comment:15 Changed 5 years ago by Jacob

milestone: 1.3

Milestone 1.3 deleted

comment:11 Changed 5 years ago by Aymeric Augustin

UI/UX: unset

Change UI/UX from NULL to False.

comment:12 Changed 4 years ago by Aymeric Augustin

The link above now redirects to GitHub issues. It looks like the Transifex devs abandonned their Trac, including the issue we had posted there.

comment:13 Changed 3 years ago by anonymous

Summary: Fix Slovenian translation plural-formsSlovenian translation plural-forms

comment:14 Changed 2 years ago by Tim Graham

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