Comment 2 for bug 1341178

Revision history for this message
Christian Boltz (cboltz) wrote :

Found it - save_profiles does:

    changed_list = sorted(changed.keys())
[...]
                if ans == 'CMD_SAVE_SELECTED':
                    profile_name = list(changed.keys())[arg]
                    write_profile_ui_feedback(profile_name)
Note that write_profile_ui_feedback() calls write_profile(), which does "changed.pop(profile_name)"
[...]
            for profile_name in changed_list:
                write_profile_ui_feedback(profile_name)

In other words: when finally saving the remaining profiles, the outdated changed_list is used.

The solution is simple:
- for profile_name in changed_list:
+ for profile_name in sorted(changed.keys()):
                 write_profile_ui_feedback(profile_name)

I'll submit the patch to the ML ;-)