From 24c51481dce81c9530056a32df4ccc4a13ebaa65 Mon Sep 17 00:00:00 2001 From: Marcus Stoegbauer Date: Wed, 3 Jul 2019 23:05:04 +0200 Subject: [PATCH] catch get exception in cfgfile, not in other places --- Userconfig/cfgfile.py | 6 ++++-- userconfig.py | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/Userconfig/cfgfile.py b/Userconfig/cfgfile.py index 70ece34..d0038ef 100644 --- a/Userconfig/cfgfile.py +++ b/Userconfig/cfgfile.py @@ -11,7 +11,6 @@ import configparser import os import re - class Conf(object): confobj = configparser.RawConfigParser() cfgfile = '' @@ -57,7 +56,10 @@ class Conf(object): """ if not self.cfgfile: raise Exception('No config file set') - return self.confobj.get(section, option) + try: + return self.confobj.get(section, option) + except configparser.NoOptionError: + raise ValueError('Option does not exist') def set(self, section, option, value): """docstring for update""" diff --git a/userconfig.py b/userconfig.py index 84f6255..04ce271 100755 --- a/userconfig.py +++ b/userconfig.py @@ -85,7 +85,7 @@ def workdir(directory): reverse_sort = False try: reverse_sort = (dir_config.get("Main", "reverse") == 'True') - except NoOptionError: + except ValueError: reverse_sort = False