datetime: Translate timezone names

Both Cities and Regions need to be translated. Also make sure that
we sort them properly, and use symbolic names for COLUMNS in the
region liststore.

https://bugzilla.gnome.org/show_bug.cgi?id=630264

configure.ac

@@ -327,6 +327,7 @@ panels/background/Makefile
 panels/background/gnome-background-panel.desktop.in
 panels/datetime/Makefile
 panels/datetime/gnome-datetime-panel.desktop.in
+panels/datetime/po-timezones/Makefile
 panels/default-applications/Makefile
 panels/default-applications/gnome-at-commandline.in
 panels/default-applications/gnome-at-session.desktop.in

panels/datetime/Makefile.am

@@ -1,6 +1,8 @@
 # This is used in GNOMECC_CAPPLETS_CFLAGS
 cappletname = datetime
 
+SUBDIRS = po-timezones
+
 icons16dir = $(datadir)/icons/hicolor/16x16/apps
 dist_icons16_DATA = icons/16x16/preferences-system-time.png
 icons22dir = $(datadir)/icons/hicolor/22x22/apps

panels/datetime/cc-datetime-panel.c

@@ -26,6 +26,9 @@
 #include <gsettings-desktop-schemas/gdesktop-enums.h>
 #include <string.h>
 #include <stdlib.h>
+#include <libintl.h>
+
+#define GETTEXT_PACKAGE_TIMEZONES GETTEXT_PACKAGE "-timezones"
 
 G_DEFINE_DYNAMIC_TYPE (CcDateTimePanel, cc_date_time_panel, CC_TYPE_PANEL)
 
@@ -35,10 +38,18 @@ G_DEFINE_DYNAMIC_TYPE (CcDateTimePanel, cc_date_time_panel, CC_TYPE_PANEL)
 enum {
   CITY_COL_CITY,
   CITY_COL_REGION,
+  CITY_COL_CITY_TRANSLATED,
+  CITY_COL_REGION_TRANSLATED,
   CITY_COL_ZONE,
   CITY_NUM_COLS
 };
 
+enum {
+  REGION_COL_REGION,
+  REGION_COL_REGION_TRANSLATED,
+  REGION_NUM_COLS
+};
+
 #define W(x) (GtkWidget*) gtk_builder_get_object (priv->builder, x)
 
 #define CLOCK_SCHEMA "org.gnome.desktop.interface"
@@ -317,7 +328,7 @@ location_changed_cb (CcTimezoneMap   *map,
     {
       gchar *string;
 
-      gtk_tree_model_get (model, &iter, 0, &string, -1);
+      gtk_tree_model_get (model, &iter, CITY_COL_CITY, &string, -1);
 
       if (!g_strcmp0 (string, split[0]))
         {
@@ -340,7 +351,7 @@ location_changed_cb (CcTimezoneMap   *map,
     {
       gchar *string;
 
-      gtk_tree_model_get (model, &iter, 0, &string, -1);
+      gtk_tree_model_get (model, &iter, CITY_COL_CITY, &string, -1);
 
       if (!g_strcmp0 (string, split[1]))
         {
@@ -374,32 +385,53 @@ struct get_region_data
   GHashTable *table;
 };
 
+/* Slash look-alikes that might be used in translations */
+#define TRANSLATION_SPLIT                                                        \
+        "\x20\x44"        /* FRACTION SLASH */                                   \
+        "\x22\x15"        /* DIVISION SLASH */                                   \
+        "\x29\xF8"        /* BIG SOLIDUS */                                      \
+        "\xFF\x0F"        /* FULLWIDTH SOLIDUS */                                \
+        "/"
+
 static void
 get_regions (TzLocation             *loc,
              struct get_region_data *data)
 {
   gchar **split;
+  gchar **split_translated;
 
   split = g_strsplit (loc->zone, "/", 2);
 
   /* remove underscores */
   g_strdelimit (split[1], "_", ' ');
 
+  /* Load the translation for it */
+  split_translated = g_strsplit_set (dgettext (GETTEXT_PACKAGE_TIMEZONES, loc->zone),
+                                     TRANSLATION_SPLIT, 2);
+
+  /* remove underscores */
+  g_strdelimit (split_translated[1], "_", ' ');
+
   if (!g_hash_table_lookup_extended (data->table, split[0], NULL, NULL))
     {
       g_hash_table_insert (data->table, g_strdup (split[0]),
                            GINT_TO_POINTER (1));
-      gtk_list_store_insert_with_values (data->region_store, NULL, 0, 0,
-                                         split[0], -1);
+      gtk_list_store_insert_with_values (data->region_store, NULL, 0,
+                                         REGION_COL_REGION, split[0],
+                                         REGION_COL_REGION_TRANSLATED, split_translated[0], -1);
     }
 
+
   gtk_list_store_insert_with_values (data->city_store, NULL, 0,
                                      CITY_COL_CITY, split[1],
+                                     CITY_COL_CITY_TRANSLATED, split_translated[1],
                                      CITY_COL_REGION, split[0],
+                                     CITY_COL_REGION_TRANSLATED, split_translated[0],
                                      CITY_COL_ZONE, loc->zone,
                                      -1);
 
   g_strfreev (split);
+  g_strfreev (split_translated);
 }
 
 static gboolean
@@ -457,7 +489,8 @@ load_regions_model (GtkListStore *regions, GtkListStore *cities)
   tz_db_free (db);
 
   /* sort the models */
-  gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE (regions), 0,
+  gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE (regions),
+                                        REGION_COL_REGION_TRANSLATED,
                                         GTK_SORT_ASCENDING);
 }
 
@@ -671,7 +704,7 @@ cc_date_time_panel_init (CcDateTimePanel *self)
                     city_modelfilter);
 
   city_modelsort = GTK_TREE_MODEL_SORT (gtk_builder_get_object (priv->builder, "city-modelsort"));
-  gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE (city_modelsort), 0,
+  gtk_tree_sortable_set_sort_column_id (GTK_TREE_SORTABLE (city_modelsort), CITY_COL_CITY_TRANSLATED,
                                         GTK_SORT_ASCENDING);
 
   gtk_tree_model_filter_set_visible_func (city_modelfilter,

panels/datetime/datetime-module.c

@@ -25,12 +25,17 @@
 
 #include <glib/gi18n-lib.h>
 
+#define GETTEXT_PACKAGE_TIMEZONES GETTEXT_PACKAGE "-timezones"
+
 void
 g_io_module_load (GIOModule *module)
 {
   bindtextdomain (GETTEXT_PACKAGE, GNOMELOCALEDIR);
   bind_textdomain_codeset (GETTEXT_PACKAGE, "UTF-8");
 
+  bindtextdomain (GETTEXT_PACKAGE_TIMEZONES, GNOMELOCALEDIR);
+  bind_textdomain_codeset (GETTEXT_PACKAGE_TIMEZONES, "UTF-8");
+
   /* register the panel */
   cc_date_time_panel_register (module);
 }

panels/datetime/datetime.ui

@@ -59,7 +59,7 @@
                             <child>
                               <object class="GtkCellRendererText" id="cellrenderertext1"/>
                               <attributes>
-                                <attribute name="text">0</attribute>
+                                <attribute name="text">1</attribute>
                               </attributes>
                             </child>
                           </object>
@@ -81,11 +81,11 @@
                         <child>
                           <object class="GtkComboBox" id="city_combobox">
                             <property name="visible">True</property>
-                            <property name="model">city-modelfilter</property>
+                            <property name="model">city-modelsort</property>
                             <child>
                               <object class="GtkCellRendererText" id="cellrenderertext2"/>
                               <attributes>
-                                <attribute name="text">0</attribute>
+                                <attribute name="text">2</attribute>
                               </attributes>
                             </child>
                           </object>
@@ -438,6 +438,8 @@
     <columns>
       <!-- column-name region -->
       <column type="gchararray"/>
+      <!-- column-name region-translated -->
+      <column type="gchararray"/>
     </columns>
   </object>
   <object class="GtkListStore" id="city-liststore">
@@ -446,6 +448,10 @@
       <column type="gchararray"/>
       <!-- column-name region -->
       <column type="gchararray"/>
+      <!-- column-name city-translated -->
+      <column type="gchararray"/>
+      <!-- column-name region-translated -->
+      <column type="gchararray"/>
       <!-- column-name zone -->
       <column type="gchararray"/>
     </columns>

panels/datetime/po-timezones/Makefile.am

@@ -0,0 +1,25 @@
+CATALOGS = $(patsubst %.po,%.mo,$(wildcard *.po))
+
+all: $(CATALOGS)
+
+NLSPACKAGE = $(GETTEXT_PACKAGE)-timezones
+
+%.mo: %.po
+	$(AM_V_GEN) msgfmt -o $@ $<
+
+install: $(CATALOGS)
+	@mkdir -p $(DESTDIR)$(localedir)
+	for n in $(CATALOGS); do \
+		l=`basename $$n .mo`; \
+		$(INSTALL) -m 755 -d $(DESTDIR)$(localedir)/$$l; \
+		$(INSTALL) -m 755 -d $(DESTDIR)$(localedir)/$$l/LC_MESSAGES; \
+		if [ -f $$n ]; then \
+			$(INSTALL) -m 644 $$n $(DESTDIR)$(localedir)/$$l/LC_MESSAGES/$(NLSPACKAGE).mo; \
+		fi; \
+	done
+
+EXTRA_DIST=$(wildcard *.po)
+
+CLEANFILES = $(CATALOGS)
+
+-include $(top_srcdir)/git.mk

panels/datetime/po-timezones/README

@@ -0,0 +1,3 @@
+Those translations are copied from system-config-date
+http://git.fedorahosted.org/git/?p=system-config-date.git;a=tree;f=po/timezones
+and should not be modified by the GNOME translation teams.