config: BaseOption.set now also set parser option value
This commit is contained in:
parent
8438f95e68
commit
e058d315d4
1 changed files with 18 additions and 12 deletions
|
@ -90,12 +90,15 @@ class BaseOption:
|
|||
|
||||
def set(self, value):
|
||||
""" Set option value to config file """
|
||||
assert self.config.config_parser, "Can set option value only if configuration file is configured"
|
||||
assert self.config.config_parser or (self.config.options and not self.no_arg), (
|
||||
"Can't set option value: configuration file not configured, not options (or no argument)")
|
||||
if value == '':
|
||||
value = None
|
||||
|
||||
if self.config.config_parser:
|
||||
if value == self.default or value is None:
|
||||
# Remove option from config
|
||||
# Remove option from config (is section exists)
|
||||
if self.config.config_parser.has_section(self.section.name):
|
||||
self.config.config_parser.remove_option(
|
||||
self.section.name, self.name)
|
||||
else:
|
||||
|
@ -107,6 +110,9 @@ class BaseOption:
|
|||
self.section.name, self.name, self.to_config(value)
|
||||
)
|
||||
|
||||
if self.config.options and not self.no_arg:
|
||||
setattr(self.config.options, self.parser_dest, value)
|
||||
|
||||
@property
|
||||
def parser_action(self):
|
||||
""" Get action as accept by argparse.ArgumentParser """
|
||||
|
|
Loading…
Reference in a new issue