Merge lp:~huxuan/gtg/port-to-configparser into lp:~gtg/gtg/old-trunk

[Xuan (Sean) Hu]

Port from configobj to ConfigParser
Cause configobj do not have an python3 port
It should be better to use official module instead third party one.

[lp:~huxuan/gtg/port-to-configparser]

1321. By Xuan (Sean) Hu on 2013-08-17

Remove configobj related description in README

[Izidor Matušov (izidor)]

The code looks good (no obvious issues there). I found couple of regressions. With the following data in tmp/ folder, GTG burns 100% of CPU. It might be caused by trying to open an nonexisting tasks (however, it shouldn't):

https://www.dropbox.com/s/589ojjkz7qsk8bf/test-config.zip

It might be because of bad serialization of lists. You don't do it the same way as we were discussing it before. You generate

opened_tasks = ["'8@1'", " '1@1'", " '6@1'", " '4@1'", '7@1']

instead of previous

opened_tasks = 8@1,1@1,6@1,4@1,7@1

Please, fix it.

review: Needs Fixing

[lp:~huxuan/gtg/port-to-configparser]

1322. By Xuan (Sean) Hu on 2013-08-19

fix opened_tasks bug

1323. By Xuan (Sean) Hu on 2013-08-19

Make SubConfig.set_list more elegent

1324. By Xuan (Sean) Hu on 2013-08-19

Make collapsed_tasks, expanded_tags actually work

1325. By Xuan (Sean) Hu on 2013-08-19

Bug fix for SubConfig.get list value

1326. By Xuan (Sean) Hu on 2013-08-19

custom_colors config should works fine now

1327. By Xuan (Sean) Hu on 2013-08-19

plugin enabled/disabled config should works fine now

[Xuan (Sean) Hu (huxuan)]

Hi, izidor, Thx for your code review.

The problem is caused by the native support for list as value for configuration in configobj, but ConfigParser make them all string. I tried to make all the list configuration to be save as '.'.join(list) pattern string and I am not sure whether it will cause more problems which I have ignored except for the collapsed_tasks, expanded_tags, custom_colors etc.

If you find any more problems, feel free to tell me. :-)

[lp:~huxuan/gtg/port-to-configparser]

1328. By Xuan (Sean) Hu on 2013-08-20

Combine SubConfig.set/set_list

[Izidor Matušov (izidor)]

It works now for me. Another really important thing to consider is the backwards compatibility. Please, make sure that you can read the same config files as the old config parser.

Steps to reproduce bug:

1, Open GTG with old config parser
2, Open windows, change width of tag sidebar and other configuration
3, Quit GTG (CTRL + Q)
4, Copy tmp/ to GTG with the new parser
5, Launch it without any problem

I get the following tracebacks:

Traceback (most recent call last):
  File "/home/izidor/projects/gtg/trunk/GTG/gtk/browser/browser.py", line 671, in on_sidebar_toggled
    self.init_tags_sidebar()
  File "/home/izidor/projects/gtg/trunk/GTG/gtk/browser/browser.py", line 221, in init_tags_sidebar
    self.tagtreeview.expand_node(path)
  File "/usr/lib/python2.7/dist-packages/liblarch_gtk/__init__.py", line 227, in expand_node
    self.collapse_node(llpath,collapsing_method=self.expand_one_row)
  File "/usr/lib/python2.7/dist-packages/liblarch_gtk/__init__.py", line 246, in collapse_node
    node_id = llpath[-1].strip("'")
IndexError: tuple index out of range
Traceback (most recent call last):
  File "./gtg", line 89, in <module>
    main()
  File "./gtg", line 85, in main
    sys.exit(gtg.main(options, args))
  File "/home/izidor/projects/gtg/trunk/GTG/gtg.py", line 123, in main
    manager = Manager(req)
  File "/home/izidor/projects/gtg/trunk/GTG/gtk/manager.py", line 75, in __init__
    self.browser = TaskBrowser(self.req, self)
  File "/home/izidor/projects/gtg/trunk/GTG/gtk/browser/browser.py", line 125, in __init__
    self.restore_state_from_conf()
  File "/home/izidor/projects/gtg/trunk/GTG/gtk/browser/browser.py", line 521, in restore_state_from_conf
    self.vtree_panes['active'].collapse_node(path)
  File "/usr/lib/python2.7/dist-packages/liblarch_gtk/__init__.py", line 246, in collapse_node
    node_id = llpath[-1].strip("'")
IndexError: tuple index out of range

Old gtg.conf file:
[browser]
contents_preview_enable = False
bg_color_enable = True
width = 425
height = 400
max = False
x_pos = 458
y_pos = 188
tag_pane = True
sidebar_width = 151
closed_task_pane = False
bottom_pane_position = 300
toolbar = True
quick_add = True
tasklist_sort_column = 5
tasklist_sort_order = 1
collapsed_tasks = ,
view = default
opened_tasks = 0@1, 5@1
font_name =
expanded_tags = ,

[plugins]
disabled = not_today, tomboy, notification_area, task_reaper, send_email, bugzilla, export, hamster, geolocalized_tasks, urgency_color, untouched_tasks
enabled = ,

review: Needs Fixing

[lp:~huxuan/gtg/port-to-configparser]

1329. By Xuan (Sean) Hu on 2013-08-22

Bug fix for backwards compatibility

[Xuan (Sean) Hu (huxuan)]

Sorry for the silly error, I do have noticed it but forget to fix it. It should work now. If there is any more problems, feel free to tell me. :-)

[lp:~huxuan/gtg/port-to-configparser]

1330. By Xuan (Sean) Hu on 2013-08-25

bug fix, mostly related to TaskConfig

[Xuan (Sean) Hu (huxuan)]

Hi, Izidor

Is there still any bugs about this ConfigParser port? I think I should continue python3 versin of GTG port based on this merge.

[Izidor Matušov (izidor)]

It looks good now :)

review: Approve

=== modified file 'GTG/backends/genericbackend.py'
--- GTG/backends/genericbackend.py	2013-02-25 08:12:02 +0000
+++ GTG/backends/genericbackend.py	2013-08-25 20:11:23 +0000
@@ -205,7 +205,7 @@
     # NOTE: for now I'm disabling changing the default backend. Once it's all
     #      set up, we will see about that (invernizzi)
     KEY_DEFAULT_BACKEND = "Default"
-    KEY_ENABLED = "Enabled"
+    KEY_ENABLED = "enabled"
     KEY_HUMAN_NAME = BACKEND_HUMAN_NAME
     KEY_ATTACHED_TAGS = "attached-tags"
     KEY_USER = "user"

=== modified file 'GTG/core/__init__.py'
--- GTG/core/__init__.py	2013-02-25 07:35:07 +0000
+++ GTG/core/__init__.py	2013-08-25 20:11:23 +0000
@@ -36,7 +36,7 @@
 """
 
 #=== IMPORT ===================================================================
-from configobj import ConfigObj, ConfigObjError
+import ConfigParser
 from xdg.BaseDirectory import xdg_data_home, xdg_config_home, xdg_data_dirs
 import os
 
@@ -46,36 +46,40 @@
 
 DEFAULTS = {
     'browser': {
-    "bg_color_enable": True,
-    "contents_preview_enable": False,
-    'tag_pane': False,
-    "sidebar_width": 120,
-    "closed_task_pane": False,
-    'bottom_pane_position': 300,
-    'toolbar': True,
-    'quick_add': True,
-    'collapsed_tasks': [],
-    'expanded_tags': [],
-    'view': 'default',
-            "opened_tasks": [],
-            'width': 400,
-            'height': 400,
-            'max': False,
-            'x_pos': 10,
-            'y_pos': 10,
-            'tasklist_sort_column': 5,
-            'tasklist_sort_order': 1,
-            "font_name": "",
-    },
-'tag_editor': {
-    "custom_colors": [],
-}
-}
-
-
-# Instead of accessing directly the ConfigObj dic, each module will have
+        "bg_color_enable": True,
+        "contents_preview_enable": False,
+        'tag_pane': False,
+        "sidebar_width": 120,
+        "closed_task_pane": False,
+        'bottom_pane_position': 300,
+        'toolbar': True,
+        'quick_add': True,
+        'collapsed_tasks': [],
+        'expanded_tags': [],
+        'view': 'default',
+        "opened_tasks": [],
+        'width': 400,
+        'height': 400,
+        'max': False,
+        'x_pos': 10,
+        'y_pos': 10,
+        'tasklist_sort_column': 5,
+        'tasklist_sort_order': 1,
+        "font_name": "",
+    },
+    'tag_editor': {
+        "custom_colors": [],
+    },
+    'plugins': {
+        "enabled": [],
+        "disabled": [],
+    }
+}
+
+
+# Instead of accessing directly the ConfigParser, each module will have
 # one SubConfig object. (one SubConfig object always match one first level
-# element of the ConfigObj directory)
+# element of the ConfigParser directory)
 #
 # The goal of the SubConfig object is to handle default value and converting
 # String to Bool and Int when needed.
@@ -88,47 +92,88 @@
 
 class SubConfig():
 
-    def __init__(self, name, conf_dic):
-        self.__name = name
-        self.__conf = conf_dic
-        if name in DEFAULTS:
-            self.__defaults = DEFAULTS[name]
-        else:
-            self.__defaults = {}
+    def __init__(self, section, conf, conf_path):
+        self._section = section
+        self._conf = conf
+        self._conf_path = conf_path
 
     # This return the value of the setting (or the default one)
     #
     # If a default value exists and is a Int or a Bool, the returned
     # value is converted to that type.
-    def get(self, name):
-        if name in self.__conf:
-            toreturn = self.__conf[name]
+    def get(self, option):
+        if self._conf.has_option(self._section, option):
+            toreturn = self._conf.get(self._section, option)
             # Converting to the good type
-            if name in self.__defaults:
-                ntype = type(self.__defaults[name])
+            if option in DEFAULTS[self._section]:
+                ntype = type(DEFAULTS[self._section][option])
                 if ntype == int:
                     toreturn = int(toreturn)
+                elif ntype == list:
+                    # All list config should be saved in ','.join(list) pattern
+                    # This is just for backward compatibility
+                    if toreturn and toreturn[0] == '[' and toreturn[-1] == ']':
+                        toreturn = toreturn[1:-1]
+                    toreturn = toreturn.split(',')
+                    while toreturn and toreturn[-1] == '':
+                        toreturn = toreturn[:-1]
                 elif ntype == bool and type(toreturn) == str:
                     toreturn = toreturn.lower() == "true"
-        elif name in self.__defaults:
-            toreturn = self.__defaults[name]
-            self.__conf[name] = toreturn
+        elif option in DEFAULTS[self._section]:
+            toreturn = DEFAULTS[self._section][option]
+            self.set(option, toreturn)
         else:
             print "Warning : no default conf value for %s in %s" % (
-                name, self.__name)
+                option, self._section)
             toreturn = None
         return toreturn
 
-    def set(self, name, value):
-        self.__conf[name] = str(value)
-        # Save immediately
-        self.__conf.parent.write()
-
-    def set_lst(self, name, value_lst):
-        self.__conf[name] = [str(s) for s in value_lst]
-        # Save immediately
-        self.__conf.parent.write()
-
+    def clear(self):
+        for option in self._conf.options(self._section):
+            self._conf.remove_option(self._section, option)
+
+    def save(self):
+        self._conf.write(open(self._conf_path, 'w'))
+
+    def set(self, option, value):
+        if type(value) == list:
+            value = ','.join(value)
+        self._conf.set(self._section, option, str(value))
+        # Save immediately
+        self.save()
+
+class TaskConfig():
+    """ TaskConfig is used to save the position and size of each task, both of
+    value are one tuple with two numbers, so set and get will use join and split
+    """
+
+    def __init__(self, conf, conf_path):
+        self._conf = conf
+        self._conf_path = conf_path
+
+    def has_section(self, section):
+        return self._conf.has_section(section)
+
+    def has_option(self, section, option):
+        return self._conf.has_option(section, option)
+
+    def add_section(self, section):
+        self._conf.add_section(section)
+
+    def get(self, tid, option):
+        value = self._conf.get(tid, option)
+        # Check single quote for backward compatibility
+        if value[0] == '(' and value[-1] == ')':
+            value = value[1:-1]
+        return value.split(', ')
+
+    def set(self, tid, option, value):
+        value = ','.join(str(x) for x in value)
+        self._conf.set(tid, option, value)
+        self.save()
+
+    def save(self):
+        self._conf.write(open(self._conf_path, 'w'))
 
 class CoreConfig(Borg):
     # The projects and tasks are of course DATA !
@@ -147,15 +192,14 @@
     SEP_TAG = "gtg-tags-sep"
     SEARCH_TAG = "search"
 
-    def check_config_file(self, file_path):
+    def check_config_file(self, path):
         """ This function bypasses the errors of config file and allows GTG
         to open smoothly"""
-        total_path = self.conf_dir + file_path
+        config = ConfigParser.ConfigParser()
         try:
-            config = ConfigObj(total_path)
-        except ConfigObjError:
-            open(total_path, "w").close()
-            config = ConfigObj(total_path)
+            config.read(path)
+        except ConfigParser.Error:
+            open(path, "w").close()
         return config
 
     def __init__(self):
@@ -173,28 +217,32 @@
             os.makedirs(self.conf_dir)
         if not os.path.exists(self.data_dir):
             os.makedirs(self.data_dir)
-        if not os.path.exists(self.conf_dir + self.CONF_FILE):
-            open(self.conf_dir + self.CONF_FILE, "w").close()
-        if not os.path.exists(self.conf_dir + self.TASK_CONF_FILE):
-            open(self.conf_dir + self.TASK_CONF_FILE, "w").close()
-        for conf_file in [self.conf_dir + self.CONF_FILE,
-                          self.conf_dir + self.TASK_CONF_FILE]:
+        self.conf_path = os.path.join(self.conf_dir, self.CONF_FILE)
+        self.task_conf_path = os.path.join(self.conf_dir, self.TASK_CONF_FILE)
+        if not os.path.exists(self.conf_path):
+            open(self.conf_path, "w").close()
+        if not os.path.exists(self.task_conf_path):
+            open(self.task_conf_path, "w").close()
+        for conf_file in [self.conf_path, self.task_conf_path]:
             if not os.access(conf_file, os.R_OK | os.W_OK):
                 raise Exception("File " + file +
                                 " is a configuration file for gtg, but it "
                                 "cannot be read or written. Please check it")
-        self.conf_dict = self.check_config_file(self.CONF_FILE)
-        self.task_conf_dict = self.check_config_file(self.TASK_CONF_FILE)
+        self._conf = self.check_config_file(self.conf_path)
+        self._task_conf = self.check_config_file(self.task_conf_path)
 
     def save(self):
         ''' Saves the configuration of CoreConfig '''
-        self.conf_dict.write()
-        self.task_conf_dict.write()
-
-    def get_subconfig(self, name):
-        if not name in self.conf_dict:
-            self.conf_dict[name] = {}
-        return SubConfig(name, self.conf_dict[name])
+        self._conf.write(open(self.conf_path, 'w'))
+        self._task_conf.write(open(self.task_conf_path, 'w'))
+
+    def get_subconfig(self, section):
+        if not self._conf.has_section(section):
+            self._conf.add_section(section)
+        return SubConfig(section, self._conf, self.conf_path)
+
+    def get_taskconfig(self):
+        return TaskConfig(self._task_conf, self.task_conf_path)
 
     def get_icons_directories(self):
         """ Returns the directories containing the icons """
@@ -214,3 +262,5 @@
 
     def set_conf_dir(self, path):
         self.conf_dir = path
+        self.conf_path = os.path.join(self.conf_dir, self.CONF_FILE)
+        self.task_conf_path = os.path.join(self.conf_dir, self.TASK_CONF_FILE)

=== modified file 'GTG/core/plugins/engine.py'
--- GTG/core/plugins/engine.py	2013-02-25 07:35:07 +0000
+++ GTG/core/plugins/engine.py	2013-08-25 20:11:23 +0000
@@ -19,8 +19,8 @@
 import imp
 import os
 import types
+import ConfigParser
 
-from configobj import ConfigObj
 import dbus
 
 from GTG.tools.borg import Borg
@@ -42,16 +42,16 @@
     missing_dbus = []
 
     def __init__(self, info, module_path):
-        """Initialize the Plugin using a ConfigObj."""
+        """Initialize the Plugin using a ConfigParser."""
         info_fields = {
-            'module_name': 'Module',
-            'full_name': 'Name',
-            'version': 'Version',
-            'authors': 'Authors',
-            'short_description': 'Short-description',
-            'description': 'Description',
-            'module_depends': 'Dependencies',
-            'dbus_depends': 'Dbus-dependencies',
+            'module_name': 'module',
+            'full_name': 'name',
+            'version': 'version',
+            'authors': 'authors',
+            'short_description': 'short-description',
+            'description': 'description',
+            'module_depends': 'dependencies',
+            'dbus_depends': 'dbus-dependencies',
         }
         for attr, field in info_fields.iteritems():
             try:
@@ -59,7 +59,12 @@
             except KeyError:
                 setattr(self, attr, [])
         # turn the enabled attribute into a bool
-        self.enabled = info['Enabled'].lower() == "true"
+        self.enabled = info['enabled'].lower() == "true"
+        # ensure the module dependencies are a list
+        if isinstance(self.module_depends, str):
+            self.module_depends = self.module_depends.split(',')
+            if not self.module_depends[-1]:
+                self.module_depends = self.module_depends[:-1]
         # ensure the dbus dependencies are a list
         if isinstance(self.dbus_depends, str):
             self.dbus_depends = [self.dbus_depends]
@@ -164,8 +169,10 @@
             for f in os.listdir(path):
                 info_file = os.path.join(path, f)
                 if os.path.isfile(info_file) and f.endswith('.gtg-plugin'):
-                    info = ConfigObj(info_file)
-                    p = Plugin(info["GTG Plugin"], self.plugin_path)
+                    info = ConfigParser.ConfigParser()
+                    info.read(info_file)
+                    info = dict(info.items("GTG Plugin"))
+                    p = Plugin(info, self.plugin_path)
                     self.plugins[p.module_name] = p
 
     def get_plugin(self, module_name):

=== modified file 'GTG/gtk/browser/browser.py'
--- GTG/gtk/browser/browser.py	2013-02-25 08:29:31 +0000
+++ GTG/gtk/browser/browser.py	2013-08-25 20:11:23 +0000
@@ -773,21 +773,25 @@
         # restore expanded state of subnodes
         self.vtree_panes['active'].get_model().foreach(
             self._expand_not_collapsed, colt)
+        self.config.set("collapsed_tasks", colt)
 
     def on_task_collapsed(self, sender, tid):
         colt = self.config.get("collapsed_tasks")
         if tid not in colt:
             colt.append(str(tid))
+        self.config.set("collapsed_tasks", colt)
 
     def on_tag_expanded(self, sender, tag):
         colt = self.config.get("expanded_tags")
         if tag not in colt:
             colt.append(tag)
+        self.config.set("expanded_tags", colt)
 
     def on_tag_collapsed(self, sender, tag):
         colt = self.config.get("expanded_tags")
         if tag in colt:
             colt.remove(str(tag))
+        self.config.set("expanded_tags", colt)
 
     def on_quickadd_activate(self, widget):
         """ Add a new task from quickadd toolbar """

=== modified file 'GTG/gtk/browser/tag_editor.py'
--- GTG/gtk/browser/tag_editor.py	2013-02-25 08:29:31 +0000
+++ GTG/gtk/browser/tag_editor.py	2013-08-25 20:11:23 +0000
@@ -305,7 +305,7 @@
         # Color selection
         self.tc_cc_colsel.unselect_color()
         # Custom colors
-        self.custom_colors = list(self.config.get('custom_colors'))
+        self.custom_colors = self.config.get('custom_colors')
         if len(self.custom_colors) > 0:
             self.tc_cc_colsel.set_custom_colors(self.custom_colors)
         # Focus
@@ -452,8 +452,7 @@
         """Callback: if a new color is added, we register it in the
         configuration"""
         self.custom_colors = self.tc_cc_colsel.get_custom_colors()
-        self.config.set_lst("custom_colors", [s for s in self.custom_colors])
-        self.req.save_config()
+        self.config.set("custom_colors", self.custom_colors)
 
     def on_close(self, widget, event, arg1=None, arg2=None, arg3=None):
         """ Callback: hide the tag editor when the close the window.

=== modified file 'GTG/gtk/editor/editor.py'
--- GTG/gtk/editor/editor.py	2013-02-25 08:29:31 +0000
+++ GTG/gtk/editor/editor.py	2013-08-25 20:11:23 +0000
@@ -50,7 +50,7 @@
         '''
         req is the requester
         vmanager is the view manager
-        taskconfig is a ConfigObj dic to save infos about tasks
+        taskconfig is a ConfigParser to save infos about tasks
         thisisnew is True when a new task is created and opened
         '''
         self.req = requester
@@ -173,14 +173,14 @@
         self.textview.grab_focus()
 
         # restoring size and position, spatial tasks
-        if self.config:
+        if self.config is not None:
             tid = self.task.get_id()
-            if tid in self.config:
-                if "position" in self.config[tid]:
-                    pos_x, pos_y = self.config[tid]["position"]
+            if self.config.has_section(tid):
+                if self.config.has_option(tid, "position"):
+                    pos_x, pos_y = self.config.get(tid, "position")
                     self.move(int(pos_x), int(pos_y))
-                if "size" in self.config[tid]:
-                    width, height = self.config[tid]["size"]
+                if self.config.has_option(tid, "size"):
+                    width, height = self.config.get(tid, "size")
                     self.window.resize(int(width), int(height))
 
         self.textview.set_editable(True)
@@ -503,7 +503,7 @@
         self.task.set_text(self.textview.get_text())
         self.task.sync()
         if self.config is not None:
-            self.config.write()
+            self.config.save()
         self.time = time.time()
     # light_save save the task without refreshing every 30seconds
     # We will reduce the time when the get_text will be in another thread
@@ -539,10 +539,10 @@
         # saving the position
         if self.config is not None:
             tid = self.task.get_id()
-            if not tid in self.config:
-                self.config[tid] = dict()
-            self.config[tid]["position"] = self.get_position()
-            self.config[tid]["size"] = self.window.get_size()
+            if not self.config.has_section(tid):
+                self.config.add_section(tid)
+            self.config.set(tid, "position", self.get_position())
+            self.config.set(tid, "size", self.window.get_size())
 
     # We define dummy variable for when close is called from a callback
     def close(self, window=None, a=None, b=None, c=None):

=== modified file 'GTG/gtk/manager.py'
--- GTG/gtk/manager.py	2013-02-25 08:29:31 +0000
+++ GTG/gtk/manager.py	2013-08-25 20:11:23 +0000
@@ -27,6 +27,7 @@
 
 import gtk
 import gobject
+import ConfigParser
 
 import GTG
 from GTG.gtk.delete_dialog import DeletionUI
@@ -50,8 +51,9 @@
     def __init__(self, req):
         self.req = req
         self.config_obj = self.req.get_global_config()
-        self.config = self.config_obj.conf_dict
-        self.task_config = self.config_obj.task_conf_dict
+        self.browser_config = self.config_obj.get_subconfig("browser")
+        self.plugins_config = self.config_obj.get_subconfig("plugins")
+        self.task_config = self.config_obj.get_taskconfig()
 
         # Editors
         self.opened_task = {}   # This is the list of tasks that are already
@@ -100,8 +102,8 @@
         self.pengine.register_api(self.plugin_api)
         # checks the conf for user settings
         try:
-            plugins_enabled = self.config["plugins"]["enabled"]
-        except KeyError:
+            plugins_enabled = self.plugins_config.get("enabled")
+        except ConfigParser.Error:
             plugins_enabled = []
         for plugin in self.pengine.get_plugins():
             plugin.enabled = plugin.module_name in plugins_enabled
@@ -180,9 +182,10 @@
             # registering as opened
             self.opened_task[uid] = tv
             # save that we opened this task
-            if uid not in self.config["browser"]["opened_tasks"]:
-                self.config["browser"]["opened_tasks"].append(uid)
-                self.config.write()
+            opened_tasks = self.browser_config.get("opened_tasks")
+            if uid not in opened_tasks:
+                opened_tasks.append(uid)
+            self.browser_config.set("opened_tasks", opened_tasks)
         return tv
 
     def close_task(self, tid):
@@ -197,9 +200,10 @@
                 # else, it close_task would be called once again
                 # by editor.close
                 editor.close()
-            if tid in self.config["browser"]["opened_tasks"]:
-                self.config["browser"]["opened_tasks"].remove(tid)
-                self.config.write()
+            opened_tasks = self.browser_config.get("opened_tasks")
+            if tid in opened_tasks:
+                opened_tasks.remove(tid)
+            self.browser_config.set("opened_tasks", opened_tasks)
         self.check_quit_condition()
 
     def check_quit_condition(self):
@@ -284,15 +288,15 @@
         for otid in self.opened_task.keys():
             open_task.append(otid)
             self.opened_task[otid].close()
-        self.config["browser"]["opened_tasks"] = open_task
+        self.browser_config.set("opened_tasks", open_task)
 
         # adds the plugin settings to the conf
         # FIXME: this code is replicated in the preference window.
         if len(self.pengine.plugins) > 0:
-            self.config["plugins"] = {}
-            self.config["plugins"]["disabled"] = \
-                [p.module_name for p in self.pengine.get_plugins("disabled")]
-            self.config["plugins"]["enabled"] = \
-                [p.module_name for p in self.pengine.get_plugins("enabled")]
+            self.plugins_config.clear()
+            self.plugins_config.set("disabled",
+                [p.module_name for p in self.pengine.get_plugins("disabled")])
+            self.plugins_config.set("enabled",
+                [p.module_name for p in self.pengine.get_plugins("enabled")])
         # plugins are deactivated
         self.pengine.deactivate_plugins()

=== modified file 'GTG/gtk/plugins.py'
--- GTG/gtk/plugins.py	2013-02-25 08:29:31 +0000
+++ GTG/gtk/plugins.py	2013-08-25 20:11:23 +0000
@@ -134,7 +134,7 @@
 
     def __init__(self, config_obj):
         self.config_obj = config_obj
-        self.config = self.config_obj.conf_dict
+        self.config = self.config_obj.get_subconfig("plugins")
         builder = gtk.Builder()
         builder.add_from_file(ViewConfig.PLUGINS_GLADE_FILE)
 
@@ -146,19 +146,12 @@
         self.plugin_depends = builder.get_object('PluginDepends')
 
         self.pengine = PluginEngine()
-        # plugin config initiation, if never used
-        if "plugins" in self.config:
-            if "enabled" not in self.config["plugins"]:
-                self.config["plugins"]["enabled"] = []
-
-            if "disabled" not in self.config["plugins"]:
-                self.config["plugins"]["disabled"] = []
-        elif self.pengine.get_plugins():
-            self.config["plugins"] = {}
-            self.config["plugins"]["disabled"] = \
-                [p.module_name for p in self.pengine.get_plugins("disabled")]
-            self.config["plugins"]["enabled"] = \
-                [p.module_name for p in self.pengine.get_plugins("enabled")]
+        # plugin config initiation
+        if self.pengine.get_plugins():
+            self.config.set("disabled",
+                [p.module_name for p in self.pengine.get_plugins("disabled")])
+            self.config.set("enabled",
+                [p.module_name for p in self.pengine.get_plugins("enabled")])
 
         # see constants PLUGINS_COL_* for column meanings
         self.plugin_store = gtk.ListStore(str, bool, str, str, bool)
@@ -255,16 +248,20 @@
         plugin = self.pengine.get_plugin(plugin_id)
         plugin.enabled = not self.plugin_store.get_value(iterator,
                                                          PLUGINS_COL_ENABLED)
+        plugins_enabled = self.config.get("enabled")
+        plugins_disabled = self.config.get("disabled")
         if plugin.enabled:
             self.pengine.activate_plugins([plugin])
-            self.config["plugins"]["enabled"].append(plugin.module_name)
-            if plugin.module_name in self.config["plugins"]["disabled"]:
-                self.config["plugins"]["disabled"].remove(plugin.module_name)
+            plugins_enabled.append(plugin.module_name)
+            if plugin.module_name in plugins_disabled:
+                plugins_disabled.remove(plugin.module_name)
         else:
             self.pengine.deactivate_plugins([plugin])
-            self.config["plugins"]["disabled"].append(plugin.module_name)
-            if plugin.module_name in self.config["plugins"]["enabled"]:
-                self.config["plugins"]["enabled"].remove(plugin.module_name)
+            plugins_disabled.append(plugin.module_name)
+            if plugin.module_name in plugins_enabled:
+                plugins_enabled.remove(plugin.module_name)
+        self.config.set("enabled", plugins_enabled)
+        self.config.set("disabled", plugins_disabled)
         self.plugin_store.set_value(iterator, PLUGINS_COL_ENABLED,
                                     plugin.enabled)
         self._update_plugin_configure(plugin)

=== modified file 'GTG/plugins/bugzilla.gtg-plugin'
--- GTG/plugins/bugzilla.gtg-plugin	2013-07-01 14:50:10 +0000
+++ GTG/plugins/bugzilla.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,12 +1,12 @@
 [GTG Plugin]
-Module=bugzilla
-Name=Bugzilla
-Short-description="Allow to link a task with a bugzilla ticket."
-Description="""Allow to link a task with a bugzilla ticket
-Just paste the URL of the ticket in the quick add entry
-and a new task will be created for this bug.
-At the moment the GNOME, Mozilla and Freedesktop.org
-bugzillas are supported."""
-Authors=Guillaume Desmottes <[e-mail]>
-Version=0.0.1
-Enabled=False
+module=bugzilla
+name=Bugzilla
+short-description=Allow to link a task with a bugzilla ticket.
+description=Allow to link a task with a bugzilla ticket
+    Just paste the URL of the ticket in the quick add entry
+    and a new task will be created for this bug.
+    At the moment the GNOME, Mozilla and Freedesktop.org
+    bugzillas are supported.
+authors=Guillaume Desmottes <[e-mail]>
+version=0.0.1
+enabled=False

=== modified file 'GTG/plugins/export.gtg-plugin'
--- GTG/plugins/export.gtg-plugin	2012-07-13 17:24:28 +0000
+++ GTG/plugins/export.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,11 +1,11 @@
 [GTG Plugin]
-Module=export
-Name=Export and print
-Short-description="Exports the tasks in the current view into a variety of formats."
-Description="""Exports the tasks in the current view into
-a variety of formats. You can also personalize the format
-of your tasks by writing your own template"""
-Authors=Luca Invernizzi <[e-mail]>, Izidor Matušov <[e-mail]>
-Version=0.2
-Enabled=False
-Dependencies=Cheetah,pdflatex,pdftk,pdfjam
+module=export
+name=Export and print
+short-description=Exports the tasks in the current view into a variety of formats.
+description=Exports the tasks in the current view into
+    a variety of formats. You can also personalize the format
+    of your tasks by writing your own template
+authors=Luca Invernizzi <[e-mail]>, Izidor Matušov <[e-mail]>
+version=0.2
+enabled=False
+dependencies=Cheetah,pdflatex,pdftk,pdfjam

=== modified file 'GTG/plugins/geolocalized-tasks.gtg-plugin'
--- GTG/plugins/geolocalized-tasks.gtg-plugin	2012-05-23 08:55:31 +0000
+++ GTG/plugins/geolocalized-tasks.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,10 +1,10 @@
 [GTG Plugin]
-Module=geolocalized_tasks
-Name=Geolocalized Tasks
-Short-description="This plugin adds geolocalized tasks to GTG!."
-Description="""This plugin adds geolocalized tasks to GTG!.
-WARNING: This plugin is still under heavy development."""
-Authors=Paulo Cabido <[e-mail]>
-Version=0.1.1
-Dependencies=configobj,Geoclue,clutter,cluttergtk,champlain,champlaingtk 
-Enabled=False
+module=geolocalized_tasks
+name=Geolocalized Tasks
+short-description=This plugin adds geolocalized tasks to GTG!.
+description=This plugin adds geolocalized tasks to GTG!.
+    WARNING: This plugin is still under heavy development.
+authors=Paulo Cabido <[e-mail]>
+version=0.1.1
+dependencies=Geoclue,clutter,cluttergtk,champlain,champlaingtk
+enabled=False

=== modified file 'GTG/plugins/hamster.gtg-plugin'
--- GTG/plugins/hamster.gtg-plugin	2012-05-23 08:55:31 +0000
+++ GTG/plugins/hamster.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,10 +1,10 @@
 [GTG Plugin]
-Module=hamster
-Name=Hamster Time Tracker Integration
-Short-description="Track time of GTG task with Hamster applet."
-Description="""Adds the ability to send a task to
-the Hamster time tracking applet"""
-Authors=Kevin Mehall <[e-mail]>
-Version=0.3
-Dbus-dependencies=org.gnome.Hamster:/org/gnome/Hamster
-Enabled=False
+module=hamster
+name=Hamster Time Tracker Integration
+short-description=Track time of GTG task with Hamster applet.
+description=Adds the ability to send a task to
+    the Hamster time tracking applet
+authors=Kevin Mehall <[e-mail]>
+version=0.3
+dbus-dependencies=org.gnome.Hamster:/org/gnome/Hamster
+enabled=False

=== modified file 'GTG/plugins/not-today.gtg-plugin'
--- GTG/plugins/not-today.gtg-plugin	2012-12-18 11:55:24 +0000
+++ GTG/plugins/not-today.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,9 +1,9 @@
 [GTG Plugin]
-Module=not_today
-Name=Not Today
-Short-description="Mark task as not to do today"
-Description="""Move the start date of tasks you don't want to do today to tomorrow."""
-Authors=Lionel Dricot <[e-mail]>
-Version=0.1
-Enabled=False
-Dependencies=
+module=not_today
+name=Not Today
+short-description=Mark task as not to do today
+description=Move the start date of tasks you don't want to do today to tomorrow.
+authors=Lionel Dricot <[e-mail]>
+version=0.1
+enabled=False
+dependencies=

=== modified file 'GTG/plugins/notification-area.gtg-plugin'
--- GTG/plugins/notification-area.gtg-plugin	2012-07-18 12:09:41 +0000
+++ GTG/plugins/notification-area.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,11 +1,11 @@
 [GTG Plugin]
-Module=notification_area
-Name=Notification area
-Short-description="Adds a GTG icon to the notification area."
-Description="""Adds a GTG icon to the notification area,
-that keeps the list of the currently workable tasks.
-To start GTG minimized,  click on the 'Configure Plugin' button 
-at the bottom of this window."""
-Authors="Paulo Cabido <[e-mail]>, Luca Invernizzi <[e-mail]>, Jono Bacon <[e-mail]>, Izidor Matušov <[e-mail]>, Antonio Roquentin <[e-mail]>"
-Version=0.95
-Enabled=False
+module=notification_area
+name=Notification area
+short-description=Adds a GTG icon to the notification area.
+description=Adds a GTG icon to the notification area,
+    that keeps the list of the currently workable tasks.
+    To start GTG minimized,  click on the 'Configure Plugin' button
+    at the bottom of this window.
+authors="Paulo Cabido <[e-mail]>, Luca Invernizzi <[e-mail]>, Jono Bacon <[e-mail]>, Izidor Matušov <[e-mail]>, Antonio Roquentin <[e-mail]>"
+version=0.95
+enabled=False

=== modified file 'GTG/plugins/send-email.gtg-plugin'
--- GTG/plugins/send-email.gtg-plugin	2012-05-23 08:55:31 +0000
+++ GTG/plugins/send-email.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,10 +1,10 @@
 [GTG Plugin]
-Module=send_email
-Name=Send task via email
-Short-description="Easily send a task via email."
-Description="""Adds a button on the toolbar to send
-easily a task via email, also sends status, due_dates, tags and subtasks."""
-Authors=Luca Invernizzi <[e-mail]>, Thibault Févry <[e-mail]>
-Version=0.2.0
-Enabled=False
-Dependencies=gio, urllib
+module=send_email
+name=Send task via email
+short-description=Easily send a task via email.
+description=Adds a button on the toolbar to send
+    easily a task via email, also sends status, due_dates, tags and subtasks.
+authors=Luca Invernizzi <[e-mail]>, Thibault Févry <[e-mail]>
+version=0.2.0
+enabled=False
+dependencies=gio, urllib

=== modified file 'GTG/plugins/task-reaper.gtg-plugin'
--- GTG/plugins/task-reaper.gtg-plugin	2012-05-23 08:55:31 +0000
+++ GTG/plugins/task-reaper.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,12 +1,12 @@
 [GTG Plugin]
-Module=task_reaper
-Name=Closed tasks remover
-Short-description="Delete old tasks automatically."
-Description="""This plugin deletes the task that have been
-closed more than a few  days ago. The number of days can be
-configured in the preference menu. There is also an option 
-to run the deletion automatically."""
-Authors=Luca Invernizzi <[e-mail]>
-Version=0.1.1
-Enabled=False
-Dependencies=,
+module=task_reaper
+name=Closed tasks remover
+short-description=Delete old tasks automatically.
+description=This plugin deletes the task that have been
+    closed more than a few  days ago. The number of days can be
+    configured in the preference menu. There is also an option
+    to run the deletion automatically.
+authors=Luca Invernizzi <[e-mail]>
+version=0.1.1
+enabled=False
+dependencies=

=== modified file 'GTG/plugins/tomboy.gtg-plugin'
--- GTG/plugins/tomboy.gtg-plugin	2012-05-23 08:55:31 +0000
+++ GTG/plugins/tomboy.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,12 +1,11 @@
 [GTG Plugin]
-Module=tomboy
-Name=Tomboy/Gnote plugin
-Short-description="Enable linking to Tomboy/GNote notes."
-Description="""This plugin lets you add a link to a Tomboy (or Gnote)
-note in your tasks, or create a new one"""
-Authors=Luca Invernizzi <[e-mail]>
-Version=0.1.9
-Enabled=False
-#Dbus-dependencies=org.gnome.Tomboy:/org/gnome/Tomboy
-Dependencies=dbus,
-
+module=tomboy
+name=Tomboy/Gnote plugin
+short-description=Enable linking to Tomboy/GNote notes.
+description=This plugin lets you add a link to a Tomboy (or Gnote)
+    note in your tasks, or create a new one
+authors=Luca Invernizzi <[e-mail]>
+version=0.1.9
+enabled=False
+#dbus-dependencies=org.gnome.Tomboy:/org/gnome/Tomboy
+dependencies=dbus,

=== modified file 'GTG/plugins/untouched-tasks.gtg-plugin'
--- GTG/plugins/untouched-tasks.gtg-plugin	2012-11-05 22:15:18 +0000
+++ GTG/plugins/untouched-tasks.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,9 +1,9 @@
 [GTG Plugin]
-Module=untouched_tasks
-Name=Untouched tasks
-Short-description="Keep track of tasks you haven't touched for a while."
-Description="""Assigns tasks that you haven't touched for a while with the @untouched tag."""
-Authors=Tom Kadwill <[e-mail]>
-Version=0.0.1
-Enabled=False
-Dependencies=gio, urllib
+module=untouched_tasks
+name=Untouched tasks
+short-description=Keep track of tasks you haven't touched for a while.
+description=Assigns tasks that you haven't touched for a while with the @untouched tag.
+authors=Tom Kadwill <[e-mail]>
+version=0.0.1
+enabled=False
+dependencies=gio, urllib

=== modified file 'GTG/plugins/urgency-color.gtg-plugin'
--- GTG/plugins/urgency-color.gtg-plugin	2012-05-23 08:55:31 +0000
+++ GTG/plugins/urgency-color.gtg-plugin	2013-08-25 20:11:23 +0000
@@ -1,16 +1,15 @@
 [GTG Plugin]
-Module=urgency_color
-Name=Urgency Color
-Short-description="Task urgency color-coding"
-Description='''This plugin will calculate the urgency status of every of your currently active tasks and color-code it accordingly.
-
-Color code
-
-Assuming your settings are default:
-Red means you are within the last 30% of days between the start-date and due-date, or your task is overdue.
-Yellow means your task's start-date has passed.
-Green means your task's start-date is oncoming.
-'''
-Authors=Wolter Hellmund <[e-mail]>
-Version=0.1
-Enabled=False
+module=urgency_color
+name=Urgency Color
+short-description=Task urgency color-coding
+description=This plugin will calculate the urgency status of every of your currently active tasks and color-code it accordingly.
+
+    Color code
+
+    Assuming your settings are default:
+    Red means you are within the last 30% of days between the start-date and due-date, or your task is overdue.
+    Yellow means your task's start-date has passed.
+    Green means your task's start-date is oncoming.
+authors=Wolter Hellmund <[e-mail]>
+version=0.1
+enabled=False

=== modified file 'GTG/tests/test_backends.py'
--- GTG/tests/test_backends.py	2013-02-25 07:35:07 +0000
+++ GTG/tests/test_backends.py	2013-08-25 20:11:23 +0000
@@ -92,7 +92,7 @@
                 dic["module"] = str(domobj.getAttribute("module"))
                 dic["pid"] = str(domobj.getAttribute("pid"))
                 dic["xmlobject"] = domobj
-                dic["Enabled"] = True
+                dic["enabled"] = True
                 dic["path"] = self.taskpath
         beobj = localfile.Backend(dic)
         expectedres = True

=== modified file 'README'
--- README	2012-11-02 09:04:08 +0000
+++ README	2013-08-25 20:11:23 +0000
@@ -15,7 +15,6 @@
  * Python, version 2.6 or above
  * PyGTK
  * python-support
- * python-configobj
  * python-xdg
  * python-dbus
  * python-liblarch 
@@ -29,8 +28,7 @@
 To install the all the required packages providing the basic features on
 Debian-based systems, execute the following command:
     $ sudo apt-get install python-support python-gtk2 python-gnome2 \
-         python-glade2 python-xdg python-configobj python-dbus \
-         python-liblarch yelp
+         python-glade2 python-xdg python-dbus python-liblarch yelp
 
 To enable API documentation generation, execute this command:
     $ sudo apt-get install python-pydoctor