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)
Change History (25)
by , 14 years ago
Attachment: | django-i18n-sl-1.2.x.patch added |
---|
comment:1 by , 14 years ago
Triage Stage: | Unreviewed → Ready for checkin |
---|
comment:2 by , 14 years ago
Summary: | Updated Slovenian translation for 1.2.x → Fix Slovenian translation plural-forms |
---|---|
Triage Stage: | Ready for checkin → Accepted |
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 by , 14 years ago
Resolution: | → invalid |
---|---|
Status: | new → closed |
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 , 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 , 14 years ago
Resolution: | invalid |
---|---|
Status: | closed → reopened |
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 , 14 years ago
milestone: | → 1.3 |
---|
Yeah, we can do this, is the attached patch the one to commit?
comment:7 by , 14 years ago
Owner: | changed from | to
---|---|
Status: | reopened → new |
comment:8 by , 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.
comment:9 by , 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."
comment:10 by , 14 years ago
Cc: | added |
---|
comment:11 by , 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 , 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.
comment:13 by , 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 , 14 years ago
Type: | Uncategorized → Bug |
---|
comment:12 by , 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 , 10 years ago
Summary: | Fix Slovenian translation plural-forms → Slovenian translation plural-forms |
---|
comment:14 by , 10 years ago
Component: | Translations → Internationalization |
---|
comment:15 by , 5 years ago
Cc: | added |
---|
Claude, can you take a look? I think we can close this old ticket, because "plural-form" can be changed in Transifex.
comment:18 by , 5 years ago
With Transifex, it's hard to know for sure. If anyone knows a method to do so, please tell me!
Updated diff of django.po