diff options
author | mathieui <mathieui@mathieui.net> | 2014-10-31 18:14:56 +0100 |
---|---|---|
committer | mathieui <mathieui@mathieui.net> | 2014-10-31 18:14:56 +0100 |
commit | ea2b703bfd07d293ba9fdd85ac657275d43da2a7 (patch) | |
tree | 426f86216e965b4330911c9ca25b267ade556ce2 | |
parent | f097efd85221b30cd5293b3f7dc52ccbb6fdc05c (diff) | |
download | poezio-ea2b703bfd07d293ba9fdd85ac657275d43da2a7.tar.gz poezio-ea2b703bfd07d293ba9fdd85ac657275d43da2a7.tar.bz2 poezio-ea2b703bfd07d293ba9fdd85ac657275d43da2a7.tar.xz poezio-ea2b703bfd07d293ba9fdd85ac657275d43da2a7.zip |
Fix the default value of Config.get to None
Might fix some not-yet-seen bugs
-rw-r--r-- | src/config.py | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/src/config.py b/src/config.py index bd703c5a..1c191eb1 100644 --- a/src/config.py +++ b/src/config.py @@ -156,16 +156,19 @@ class Config(RawConfigParser): if not self.has_section(section): self.add_section(section) - def get(self, option, default='', section=DEFSECTION): + def get(self, option, default=None, section=DEFSECTION): """ get a value from the config but return a default value if it is not found The type of default defines the type returned """ - if self.default and not default \ - and self.default.get(section, {}).get(option) is not None: - default = self.default[section][option] + if default is None: + if self.default: + default = self.default.get(section, {}).get(option) + else: + default = '' + try: if type(default) == int: res = self.getint(option, section) @@ -177,6 +180,7 @@ class Config(RawConfigParser): res = self.getstr(option, section) except (NoOptionError, NoSectionError, ValueError, AttributeError): return default + if res is None: return default return res |