[Kstars-devel] KDE/kdeedu/kstars/kstars

[Akarsh Simha]

SVN commit 867116 by asimha:

Fixes incorrect message in What's Up Tonight Dialog. Should not
translate an empty string.

Many thanks to Avinash Krishnakumar and Suvinay S for helping me out.

CCMAIL: kstars-devel at kde.org



 M  +4 -4      geolocation.h  


--- trunk/KDE/kdeedu/kstars/kstars/geolocation.h #867115:867116
@@ -124,19 +124,19 @@
     QString name() const { return Name; }
     /**@return translated City name
     	*/
-    QString translatedName() const { return i18nc("City name (optional, probably does not need a translation)", Name.toUtf8().data()); }
-    /**@return untranslated Province name
+    QString translatedName() const { return Name.isEmpty() ? QString() : i18nc("City name (optional, probably does not need a translation)", Name.toUtf8().data()); }
+    /**@return untranslated Province name 
     	*/
     QString province() const { return Province; }
     /**@return translated Province name
     	*/
-    QString translatedProvince() const { return i18nc("Region/state name (optional, rarely needs a translation)", Province.toUtf8().data()); }
+    QString translatedProvince() const { return Province.isEmpty() ? QString() : i18nc("Region/state name (optional, rarely needs a translation)", Province.toUtf8().data()); }
     /**@return untranslated Country name
     	*/
     QString country() const { return Country; }
     /**@return translated Country name
      */
-    QString translatedCountry() const { return i18nc("Country name (optional, but should be translated)", Country.toUtf8().data()); }
+    QString translatedCountry() const { return Country.isEmpty() ? QString() :  i18nc("Country name (optional, but should be translated)", Country.toUtf8().data()); }
 
     /**@return comma-separated city, province, country names (each localized)
      */