Ticket #1341: dslo1341_3.patch

bin/sugar-session

@@ -164 +164 @@
 
         model = gconf_client.get_string(\
             '/desktop/sugar/peripherals/keyboard/model')
-        if model is not None:
+        if model:
             configrec.set_model(model)
 
         options = gconf_client.get_list(\

extensions/cpsection/keyboard/model.py

@@ -133 +133 @@
 
     def set_model(self, model):
         """Sets the supplied keyboard model"""
+        if model is None or not model:
+            return
         self._gconf_client.set_string(_MODEL_KEY, model)
         self._configrec.set_model(model)
         self._configrec.activate(self._engine)
@@ -150 +152 @@
 
     def set_layouts(self, layouts):
         """Sets the supplied keyboard layouts (with variants)"""
+        if layouts is None or not layouts:
+            return
         self._gconf_client.set_list(_LAYOUTS_KEY, gconf.VALUE_STRING, layouts)
         layouts_list = []
         variants_list = []

extensions/cpsection/keyboard/view.py

@@ -204 +204 @@
         kmodel_combo.add_attribute(cell, 'text', 1)
 
         self._kmodel = self._keyboard_manager.get_current_model()
-        for row in kmodel_store:
-            if self._kmodel in row[0]:
-                kmodel_combo.set_active_iter(row.iter)
-                break
+        if self._kmodel is not None:
+            for row in kmodel_store:
+                if self._kmodel in row[0]:
+                    kmodel_combo.set_active_iter(row.iter)
+                    break
 
         box_kmodel.pack_start(kmodel_combo, expand = False)
         self._vbox.pack_start(box_kmodel, expand=False)