summaryrefslogtreecommitdiff
path: root/lib/python
diff options
context:
space:
mode:
authorErovia <erovia@users.noreply.github.com>2019-11-24 20:24:47 +0100
committerFlorian Didron <fdidron@users.noreply.github.com>2020-02-26 10:15:12 +0900
commitbbcc6051955b8015e87e306b33c70e9517a7e1ca (patch)
treea38040575c2d8c6c624f86ab8c551c7743b65827 /lib/python
parent06e5032ad3e4721436a7ad29bc84a541a0ed8305 (diff)
MILC: Fix/complete attribute heritance
If an undefined attribute of a submodule is accessed, fall back to same attribute of the submodule's parent.
Diffstat (limited to 'lib/python')
-rw-r--r--lib/python/milc.py17
1 files changed, 15 insertions, 2 deletions
diff --git a/lib/python/milc.py b/lib/python/milc.py
index 4392c8376a..92b1278f4c 100644
--- a/lib/python/milc.py
+++ b/lib/python/milc.py
@@ -178,8 +178,9 @@ class ConfigurationSection(Configuration):
def __getitem__(self, key):
"""Returns a config value, pulling from the `user` section as a fallback.
+ This is called when the attribute is accessed either via the get method or through [ ] index.
"""
- if key in self._config:
+ if key in self._config and self._config[key]:
return self._config[key]
elif key in self.parent.user:
@@ -187,6 +188,15 @@ class ConfigurationSection(Configuration):
return None
+ def __getattr__(self, key):
+ """Returns the config value from the `user` section.
+ This is called when the attribute is accessed via dot notation but does not exists.
+ """
+ if key in self.parent.user:
+ return self.parent.user[key]
+
+ return None
+
def handle_store_boolean(self, *args, **kwargs):
"""Does the add_argument for action='store_boolean'.
@@ -519,7 +529,10 @@ class MILC(object):
self.config[section][argument] = arg_value
else:
if argument not in self.config[section]:
- self.config[section][argument] = getattr(self.args, argument)
+ # Check if the argument exist for this section
+ arg = getattr(self.args, argument)
+ if arg:
+ self.config[section][argument] = arg
self.release_lock()