From e7ca63c7c5a94279a284bbdcdd14e8c6c5597267 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Aur=C3=A9lien=20G=C3=A2teau?= <agateau@kde.org>
Date: Tue, 22 Apr 2014 10:54:03 +0200
Subject: Improve plural handling for code using Qt translation system

- Always load "en" translation: This way if a plural string is not translated,
  we fallback to the correct english plural form.

- Generate .ts files with correct plural settings

REVIEW: 117629
---
 modules/ECMQmLoader.cpp.in | 24 ++++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)

(limited to 'modules/ECMQmLoader.cpp.in')

diff --git a/modules/ECMQmLoader.cpp.in b/modules/ECMQmLoader.cpp.in
index fd530602..2d8531f3 100644
--- a/modules/ECMQmLoader.cpp.in
+++ b/modules/ECMQmLoader.cpp.in
@@ -10,32 +10,36 @@
 
 #include <QDebug>
 
-static QTranslator *createTranslator(const QString &localeDirName)
+static bool loadTranslation(const QString &localeDirName)
 {
     QString subPath = QStringLiteral("locale/") + localeDirName + QStringLiteral("/LC_MESSAGES/@catalog_name@.qm");
     QString fullPath = QStandardPaths::locate(QStandardPaths::GenericDataLocation, subPath);
     if (fullPath.isEmpty()) {
-        return 0;
+        return false;
     }
     QTranslator *translator = new QTranslator(QCoreApplication::instance());
     if (!translator->load(fullPath)) {
         delete translator;
-        return 0;
+        return false;
     }
-    return translator;
+    QCoreApplication::instance()->installTranslator(translator);
+    return true;
 }
 
 static void load()
 {
+    // The way Qt translation system handles plural forms makes it necessary to
+    // have a translation file which contains only plural forms for `en`. That's
+    // why we load the `en` translation unconditionally, then load the
+    // translation for the current locale to overload it.
+    loadTranslation(QStringLiteral("en"));
+
     QLocale locale = QLocale::system();
-    QTranslator *translator = createTranslator(locale.name());
-    if (!translator) {
-        translator = createTranslator(locale.bcp47Name());
-        if (!translator) {
-            return;
+    if (locale.name() != QStringLiteral("en")) {
+        if (!loadTranslation(locale.name())) {
+            loadTranslation(locale.bcp47Name());
         }
     }
-    QCoreApplication::instance()->installTranslator(translator);
 }
 
 Q_COREAPP_STARTUP_FUNCTION(load)
-- 
cgit v1.2.1