From 58d1988b6571afddbd2dc824a40ba7998ecc5757 Mon Sep 17 00:00:00 2001 From: lafricain79 Date: Sun, 2 Aug 2026 15:46:44 +0200 Subject: [PATCH] parallel sets: fix set name key conversion on set switch When switching sets via the ComboBox, the displayed name (with spaces) was being stored directly in settings.xml instead of the XML key (with underscores), causing a mismatch on next startup. Apply name_to_key() before storing parallel_set_current and before calling get_parallel_set() when switching sets. --- src/gtk/preferences_dialog.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/gtk/preferences_dialog.c b/src/gtk/preferences_dialog.c index 3922e3fa0..fe7435e92 100644 --- a/src/gtk/preferences_dialog.c +++ b/src/gtk/preferences_dialog.c @@ -3358,16 +3358,18 @@ static void on_parallel_sets_combo_changed(GtkComboBox *combo, /* the active set is saved automatically by on_parallel_reordered */ /* load the new set */ - gchar **modules = get_parallel_set(name); + gchar *key = name_to_key(name); + gchar **modules = get_parallel_set(key); + g_free(key); if (modules) { g_strfreev(settings.parallel_list); settings.parallel_list = modules; } - if (settings.parallel_set_current) g_free(settings.parallel_set_current); - settings.parallel_set_current = name; - xml_set_or_create_value("modules", "parallel_set_current", name); + settings.parallel_set_current = name_to_key(name); + xml_set_or_create_value("modules", "parallel_set_current", settings.parallel_set_current); + g_free(name); xml_save_settings_doc(settings.fnconfigure); /* refresh listview */