Bug 230057 - Fix nplurals/plural values for hungarian language
Summary: Fix nplurals/plural values for hungarian language
Status: RESOLVED FIXED
Alias: None
Product: lokalize
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Nick Shaforostoff
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-03-09 07:48 UTC by Laszlo Papp
Modified: 2010-03-15 10:43 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Laszlo Papp 2010-03-09 07:48:59 UTC
Version:            (using KDE 4.4.1)
OS:                Linux
Installed from:    Archlinux Packages

If I set the language for Hungarian in Lokalize, I do the translation, and then I save it and exit, I see it:
"Plural-Forms: nplurals=2; plural=1;\n" instead of "Plural-Forms: nplurals=1; plural=0;\n"

So it would be nice for it to contain the rights nplurals/plural values to help us, huganrian translators, if it's possible, thanks in advance.
Comment 1 Laszlo Papp 2010-03-11 01:24:30 UTC
Hello, this patch was tested and works pretty well, I've got SVN account, may I commit it ?

Index: src/catalog/gettextheader.cpp
===================================================================
--- src/catalog/gettextheader.cpp	(revision 1101812)
+++ src/catalog/gettextheader.cpp	(working copy)
@@ -8,7 +8,7 @@
   published by the Free Software Foundation; either version 2 of
   the License or (at your option) version 3 or any later version
   accepted by the membership of KDE e.V. (or its successor approved
-  by the membership of KDE e.V.), which shall act as a proxy 
+  by the membership of KDE e.V.), which shall act as a proxy
   defined in Section 14 of version 3 of the license.
 
   This program is distributed in the hope that it will be useful,
@@ -106,8 +106,8 @@
 "nplurals=3; plural=n==1 ? 0 : n==2 ? 1 : 2;",
 "nplurals=2; plural=(n != 1);",
 "nplurals=3; plural=(n%10==1 && n%100!=11 ? 0 : n%10>=2 && n%10<=4 && (n%100<10 || n%100>=20) ? 1 : 2);",
+"nplurals=1; plural=0;",
 "nplurals=2; plural=(n != 1);",
-"nplurals=2; plural=(n != 1);",
 "nplurals=1; plural=0;",
 "nplurals=1; plural=0;",
 "nplurals=3; plural=(n%10==1 && n%100!=11 ? 0 : n%10>=2 && (n%100<10 || n%100>=20) ? 1 : 2);",
Comment 2 Nick Shaforostoff 2010-03-12 09:14:10 UTC
yes, please!
Comment 3 Nick Shaforostoff 2010-03-12 09:14:53 UTC
don't forget to add a 
BUG: 230057
line to your commit message
Comment 4 Laszlo Papp 2010-03-15 10:30:08 UTC
I've committed, you can close it.