From 1ebf1c4988552263858eafc1282ef405d9393796 Mon Sep 17 00:00:00 2001 From: Russell Schmidt Date: Sun, 26 Jan 2025 16:07:04 -0600 Subject: [PATCH] Fix crash on cancelling settings input When backing out of entering user short name or long name, the app would crash. Once it didn't crash, backing out would set these fields to None, rather than cancelling the change. --- settings.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/settings.py b/settings.py index 7b67058..3ddf631 100644 --- a/settings.py +++ b/settings.py @@ -188,6 +188,7 @@ def settings_menu(stdscr, interface): if selected_option in ['longName', 'shortName', 'isLicensed']: if selected_option in ['longName', 'shortName']: new_value = get_user_input(f"Current value for {selected_option}: {current_value}") + new_value = current_value if new_value is None else new_value current_menu[selected_option] = (field, new_value) elif selected_option == 'isLicensed': @@ -233,7 +234,7 @@ def settings_menu(stdscr, interface): modified_settings[selected_option] = new_value # Convert enum string to int - if field.enum_type: + if field and field.enum_type: enum_value_descriptor = field.enum_type.values_by_number.get(new_value) new_value = enum_value_descriptor.name if enum_value_descriptor else new_value