parchlinux-mobile/parchbootstrap
1
0
Fork 0
forked from parchlinux-mobile/kupferbootstrap
Code Issues Pull requests Activity

config.prompt_config(): fix changed detection for lists due to conversion to comma-separated strings

Browse source
This commit is contained in:
InsanePrawn 2022-08-17 01:10:06 +02:00
parent e17a69ed81
commit 36b321aa2d
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
config.py
View file

@ -390,6 +390,8 @@ def prompt_config(
whether the result is different, considering empty strings and None equal to each other. whether the result is different, considering empty strings and None equal to each other.
""" """
original_default = default
def true_or_zero(to_check) -> bool: def true_or_zero(to_check) -> bool:
"""returns true if the value is truthy or int(0)""" """returns true if the value is truthy or int(0)"""
zero = 0 # compiler complains about 'is with literal' otherwise zero = 0 # compiler complains about 'is with literal' otherwise
@ -411,7 +413,7 @@ def prompt_config(
text = click.style(text, bold=True) text = click.style(text, bold=True)
result = click.prompt(text, type=field_type, default=default, value_proc=value_conv, show_default=True) # type: ignore result = click.prompt(text, type=field_type, default=default, value_proc=value_conv, show_default=True) # type: ignore
changed = (result != default) and (true_or_zero(default) or true_or_zero(result)) changed = result != (original_default if field_type == list else default) and (true_or_zero(default) or true_or_zero(result))
if changed and echo_changes: if changed and echo_changes:
print(f'value changed: "{text}" = "{result}"') print(f'value changed: "{text}" = "{result}"')