Opened 14 years ago

Last modified 5 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@…, Claude Paroz Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

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 14 years ago.
Updated diff of django.po
14129.diff (10.6 KB ) - added by Claude Paroz 14 years ago.
Only plural forms changed

Download all attachments as: .zip

Change History (25)

by Gasper Zejn, 14 years ago

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

Updated diff of django.po

comment:1 by Russell Keith-Magee, 14 years ago

Triage Stage: UnreviewedReady for checkin

comment:2 by Claude Paroz, 14 years ago

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:

http://docs.djangoproject.com/en/dev/internals/contributing/#submitting-and-maintaining-translations

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

comment:3 by Gasper Zejn, 14 years ago

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.

Thanks.

comment:4 by Claude Paroz, 14 years ago

Yes, the current equation is valid. It's just that more and more projects are using another one. Read the rationale here:
https://answers.launchpad.net/launchpad/+question/18324
See also: http://translate.sourceforge.net/wiki/l10n/pluralforms

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

comment:5 by Gasper Zejn, 14 years ago

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 by Jannis Leidel, 14 years ago

milestone: 1.3

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

by Claude Paroz, 14 years ago

Attachment: 14129.diff added

Only plural forms changed

comment:7 by Jannis Leidel, 14 years ago

Owner: changed from nobody to Jannis Leidel
Status: reopenednew

comment:8 by Jannis Leidel, 14 years ago

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.

http://trac.transifex.org/ticket/691

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

comment:9 by Jannis Leidel, 14 years ago

Okay, the Transifex developers now have tested it on test.transifex.net 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"
"Report-Msgid-Bugs-To: http://code.djangoproject.com/\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 <en@li.org>\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."

https://gist.github.com/b8b851f3c4bffa08378f

comment:10 by diegobz@…, 14 years ago

Cc: diegobz@… added

comment:11 by Gasper Zejn, 14 years ago

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]

Thanks.

comment:12 by Jannis Leidel, 14 years ago

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 14 years ago by Jannis Leidel (previous) (diff)

comment:13 by patchhammer, 14 years ago

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

14129.diff fails to apply cleanly on to trunk

comment:14 by Julien Phalip, 14 years ago

Type: UncategorizedBug

comment:15 by Jacob, 13 years ago

milestone: 1.3

Milestone 1.3 deleted

comment:11 by Aymeric Augustin, 13 years ago

UI/UX: unset

Change UI/UX from NULL to False.

comment:12 by Aymeric Augustin, 12 years ago

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 by anonymous, 10 years ago

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

comment:14 by Tim Graham, 10 years ago

Component: TranslationsInternationalization

comment:15 by Mariusz Felisiak, 5 years ago

Cc: Claude Paroz added

Claude, can you take a look? I think we can close this old ticket, because "plural-form" can be changed in Transifex.

comment:16 by Claude Paroz, 5 years ago

can be changed or can't be changed?

comment:17 by Mariusz Felisiak, 5 years ago

I think it can be changed in Transifex, am I right? Probably not :|

comment:18 by Claude Paroz, 5 years ago

With Transifex, it's hard to know for sure. If anyone knows a method to do so, please tell me!

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