]> cloud.milkyroute.net Git - dolphin.git/commitdiff
Call KLocalizedString::setApplicationDomain("dolphin");
authorAlbert Astals Cid <aacid@kde.org>
Sun, 29 Nov 2015 18:19:33 +0000 (19:19 +0100)
committerAlbert Astals Cid <aacid@kde.org>
Sun, 29 Nov 2015 18:19:33 +0000 (19:19 +0100)
dolphin is using
   -DTRANSLATION_DOMAIN=\"dolphin\"
that is wrong for applications, apps need to use
   KLocalizedString::setApplicationDomain
since that makes some other things like the transltators tab in the about
dialog show.

I have not removed -DTRANSLATION_DOMAIN=\"dolphin\" because on the other hand
this code is also a library (i.e. the dolphin kpart).

The "being pedandly correct" fix would be using -DTRANSLATION_DOMAIN=\"dolphin\"
only to compile the files that are part of the kpart but that's prone to
break at some point and using both KLocalizedString::setApplicationDomain and
-DTRANSLATION_DOMAIN does not create any problem so i went that route

CCMAIL: kfm-devel@kde.org

src/main.cpp

index 0c5ac87f2feaf0482a81b6c6f135b97bd5ad3641..f51932913ce870bd78b85ba564bda305dc6f9977 100644 (file)
@@ -48,6 +48,8 @@ extern "C" Q_DECL_EXPORT int kdemain(int argc, char **argv)
     migrate.setUiFiles(QStringList() << QStringLiteral("dolphinpart.rc") << QStringLiteral("dolphinui.rc"));
     migrate.migrate();
 
+    KLocalizedString::setApplicationDomain("dolphin");
+
     KAboutData aboutData("dolphin", i18n("Dolphin"), QStringLiteral(DOLPHIN_VERSION_STRING),
                          i18nc("@title", "File Manager"),
                          KAboutLicense::GPL,