modules/fm-orchestrator
1
0
Fork 0
mirror of https://pagure.io/fm-orchestrator.git synced 2026-02-13 10:05:15 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

convention: avoid confusion with __setitem__

Browse Source
This commit is contained in:
Filip Valder
2016-11-11 14:53:43 +01:00
parent 6d0903c872
commit 3313cbb007
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
module_build_service/config.py
View File

@@ -197,7 +197,7 @@ def from_app_config():
for key, value in app.config.items():
# lower keys
key = key.lower()
conf.setitem(key, value)
conf.set_item(key, value)
return conf
@@ -212,10 +212,10 @@ class Config(object):
in self._defaults}
for conf_item in self._defaults:
self.setitem(conf_item['name'], conf_item['default'])
self.set_item(conf_item['name'], conf_item['default'])
def setitem(self, key, value):
if key == 'setitem' or key.startswith('_'):
def set_item(self, key, value):
if key == 'set_item' or key.startswith('_'):
raise Exception("Configuration item's name is not allowed: %s" % key)
# registered defaults