From eaf139147aa953da991b04d2de03a3994faf42fd Mon Sep 17 00:00:00 2001 From: gwen Date: Tue, 10 Sep 2024 10:13:23 +0200 Subject: [PATCH] refactoring --- src/rougail/user-data-environment/data.py | 19 +++++++------------ 1 file changed, 7 insertions(+), 12 deletions(-) diff --git a/src/rougail/user-data-environment/data.py b/src/rougail/user-data-environment/data.py index 5d2f17c..e1e47b2 100644 --- a/src/rougail/user-data-environment/data.py +++ b/src/rougail/user-data-environment/data.py @@ -83,18 +83,13 @@ class RougailUserDataEnvironment: if option_name in unused_environment_var: self.unused_environment_var.remove(option_name) # let's parse the environment variables values + option_bash_value = get_rougail_environment_dict()[option_name] if ismulti: - option_bash_value = get_rougail_environment_dict()[option_name] # here we expect the bash option value of a multi to be coma separated: - option_value = option_bash_value.split(",") + # FIXME faire un strip() la dessus + option_bash_value = option_bash_value.split(",") if type_obj is not None: - option_value_typed = [type_obj(opt) for opt in option_value] - option.value.set(option_value_typed) - else: - option.value.set(option_value) - else: - option_bash_value = get_rougail_environment_dict()[option_name] - if type_obj is not None: - option.value.set(type_obj(option_bash_value)) - else: - option.value.set(option_bash_value) + option_bash_value = [type_obj(opt) for opt in option_value] + if type_obj: + option_bash_value = type_obj(option_bash_value) + option.value.set(option_bash_value)