From c58a6c801d10441565aac0bc857e92a81dc7d116 Mon Sep 17 00:00:00 2001 From: Andreas Preikschat Date: Tue, 15 Jan 2013 00:31:02 +0100 Subject: [PATCH] don't overwrite build-ins --- openlp/core/lib/settingsmanager.py | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) diff --git a/openlp/core/lib/settingsmanager.py b/openlp/core/lib/settingsmanager.py index 724f0f4de..820365054 100644 --- a/openlp/core/lib/settingsmanager.py +++ b/openlp/core/lib/settingsmanager.py @@ -82,7 +82,7 @@ class SettingsManager(object): Settings().setValue(section + u'/' + name, directory) @staticmethod - def set_list(section, name, list): + def set_list(section, name, list_to_save): """ Save a list to application settings. @@ -92,16 +92,16 @@ class SettingsManager(object): ``name`` The name of the list to save. - ``list`` + ``list_to_save`` The list of values to save. """ settings = Settings() settings.beginGroup(section) old_count = settings.value(u'%s count' % name, 0) - new_count = len(list) + new_count = len(list_to_save) settings.setValue(u'%s count' % name, new_count) for counter in range(new_count): - settings.setValue(u'%s %d' % (name, counter), list[counter - 1]) + settings.setValue(u'%s %d' % (name, counter), list_to_save[counter - 1]) if old_count > new_count: # Tidy up any old list items for counter in range(new_count, old_count): @@ -122,14 +122,14 @@ class SettingsManager(object): settings = Settings() settings.beginGroup(section) list_count = settings.value(u'%s count' % name, 0) - list = [] + loaded_list = [] if list_count: for counter in range(list_count): item = settings.value(u'%s %d' % (name, counter), u'') if item: - list.append(item) + loaded_list.append(item) settings.endGroup() - return list + return loaded_list @staticmethod def get_files(section=None, extension=None):