Handle situations where keyboard model is stored as '' or None. Compl…

…etes dslo #1341

bin/sugar-session

@@ -164,7 +164,7 @@ def setup_keyboard_cb(gconf_client):
 
         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,6 +133,8 @@ def get_max_layouts(self):
 
     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,6 +152,8 @@ def set_option_group(self, option_group):
 
     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,10 +204,11 @@ def _setup_kmodel(self):
         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)