keyboard control panel chokes on non-empty option group

[sascha_silbe]

This is the trigger for #2021.

If the option group is non-empty (e.g. because it has been configured using the regular Xorg ways), the keyboard control panel will wrap it in another list and gconf throws an exception:

1275296170.989435 TRACE root: set_option_group(<cpsection.keyboard.model.KeyboardManager object at 0x8f7e94c>, ['terminate:ctrl_alt_bksp']) invoked
1275296170.990156 DEBUG root: options=[['terminate:ctrl_alt_bksp']]
1275296170.990818 ERROR root: Exception occured in set_option_group
Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/sugar/logger.py", line 189, in _trace
    res = f(*args, **kwargs)
  File "/usr/share/sugar/extensions/cpsection/keyboard/model.py", line 154, in set_option_group
    self._gconf_client.set_list(_OPTIONS_KEY, gconf.VALUE_STRING, options)
TypeError: value should be a string
Traceback (most recent call last):
  File "/usr/lib/python2.5/site-packages/jarabe/controlpanel/gui.py", line 321, in __select_option_cb
    self.show_section_view(option)
  File "/usr/lib/python2.5/site-packages/jarabe/controlpanel/gui.py", line 218, in show_section_view
    self._options[option]['alerts'])
  File "/usr/share/sugar/extensions/cpsection/keyboard/view.py", line 179, in __init__
    self._setup_group_switch_option()
  File "/usr/share/sugar/extensions/cpsection/keyboard/view.py", line 270, in _setup_group_switch_option
    self._keyboard_manager.get_current_option_group()
  File "/usr/share/sugar/extensions/cpsection/keyboard/model.py", line 125, in get_current_option_group
    self.set_option_group(options)
  File "<string>", line 2, in set_option_group
  File "/usr/lib/python2.5/site-packages/sugar/logger.py", line 189, in _trace
    res = f(*args, **kwargs)
  File "/usr/share/sugar/extensions/cpsection/keyboard/model.py", line 154, in set_option_group
    self._gconf_client.set_list(_OPTIONS_KEY, gconf.VALUE_STRING, options)
TypeError: value should be a string

[sascha_silbe]

​Patch #99 on Patchwork.

[sayamindu]

Patch looks OK. Please commit.

[tomeu]

Pushed as ​http://git.sugarlabs.org/projects/sugar/repos/mainline/commits/a85a8b7b65

Thanks!

[dnarvaez]

Milestone 0.88.x deleted