Use RawConfigParser.get{int,bool,float} whenever possible

config.get('option', 'value').lower() == 'value' is just ugly and
stupid, especially for bool.

One if in basetabs:556 was also missing a comparison, leading to True
whenever the option was set.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/bookmark.py b/src/bookmark.py
index a68cc27a..98bfecc4 100644
--- a/src/bookmark.py
+++ b/src/bookmark.py
@@ -157,7 +157,7 @@ def save_local():
 def save(xmpp, core=None):
     """Save all the bookmarks."""
     save_local()
-    if config.get('use_remote_bookmarks', 'true').lower() != 'false':
+    if config.get('use_remote_bookmarks', True):
         preferred = config.get('use_bookmarks_method', 'privatexml')
         if not save_remote(xmpp, method=preferred) and core:
             core.information('Could not save bookmarks.', 'Error')