From 1e7214a86b7ebc05b339c3323020b1492dedcf7a Mon Sep 17 00:00:00 2001 From: Fabian Dill Date: Sat, 19 Jun 2021 01:00:41 +0200 Subject: [PATCH] fix required plando options triggering on empty string --- Mystery.py | 17 +++++++++-------- 1 file changed, 9 insertions(+), 8 deletions(-) diff --git a/Mystery.py b/Mystery.py index 947a67fb..8f36e7d7 100644 --- a/Mystery.py +++ b/Mystery.py @@ -497,15 +497,16 @@ def roll_settings(weights: dict, plando_options: typing.Set[str] = frozenset(("b raise Exception(f"Settings reports required version of generator is at least {version}, " f"however generator is of version {__version__}") required_plando_options = requirements.get("plando", "") - required_plando_options = set(option.strip() for option in required_plando_options.split(",")) - required_plando_options -= plando_options if required_plando_options: - if len(required_plando_options) == 1: - raise Exception(f"Settings reports required plando module {', '.join(required_plando_options)}, " - f"which is not enabled.") - else: - raise Exception(f"Settings reports required plando modules {', '.join(required_plando_options)}, " - f"which are not enabled.") + required_plando_options = set(option.strip() for option in required_plando_options.split(",")) + required_plando_options -= plando_options + if required_plando_options: + if len(required_plando_options) == 1: + raise Exception(f"Settings reports required plando module {', '.join(required_plando_options)}, " + f"which is not enabled.") + else: + raise Exception(f"Settings reports required plando modules {', '.join(required_plando_options)}, " + f"which are not enabled.") ret = argparse.Namespace() ret.name = get_choice('name', weights)