Bug 230057

Summary: Fix nplurals/plural values for hungarian language
Product: [Applications] lokalize Reporter: Laszlo Papp <lpapp>
Component: generalAssignee: Nick Shaforostoff <shafff>
Status: RESOLVED FIXED    
Severity: normal CC: lpapp
Priority: NOR    
Version: unspecified   
Target Milestone: ---   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In:

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.