summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorSimo Sorce <simo@redhat.com>2014-10-29 16:31:06 -0400
committerPatrick Uiterwijk <puiterwijk@redhat.com>2014-11-12 23:47:31 +0100
commit4c56c3f3801fcfbf463906cff3c8dbbb0e23ed87 (patch)
tree15e31fb2e9f533417ab199cb3a2685f33af6ec3d
parent910496f062f7a06cbd71422767cfa10bb4200075 (diff)
downloadipsilon-4c56c3f3801fcfbf463906cff3c8dbbb0e23ed87.tar.gz
ipsilon-4c56c3f3801fcfbf463906cff3c8dbbb0e23ed87.tar.xz
ipsilon-4c56c3f3801fcfbf463906cff3c8dbbb0e23ed87.zip
Make internal functions private
load_data and reset_data are used only internally, turn them into private functions Signed-off-by: Simo Sorce <simo@redhat.com> Reviewed-by: Patrick Uiterwijk <puiterwijk@redhat.com>
-rwxr-xr-xipsilon/util/data.py12
1 files changed, 6 insertions, 6 deletions
diff --git a/ipsilon/util/data.py b/ipsilon/util/data.py
index 382f76f..2af2bdc 100755
--- a/ipsilon/util/data.py
+++ b/ipsilon/util/data.py
@@ -150,7 +150,7 @@ class Store(Log):
self._row_to_dict_tree(data, r)
return data
- def load_data(self, table, columns, kvfilter=None):
+ def _load_data(self, table, columns, kvfilter=None):
rows = []
try:
q = self._query(self._db, table, columns, trans=False)
@@ -162,13 +162,13 @@ class Store(Log):
def load_config(self):
table = 'config'
columns = ['name', 'value']
- return self.load_data(table, columns)
+ return self._load_data(table, columns)
def load_options(self, table, name=None):
kvfilter = dict()
if name:
kvfilter['name'] = name
- options = self.load_data(table, OPTIONS_COLUMNS, kvfilter)
+ options = self._load_data(table, OPTIONS_COLUMNS, kvfilter)
if name and name in options:
return options[name]
return options
@@ -237,7 +237,7 @@ class Store(Log):
kvfilter['name'] = name
if value:
kvfilter['value'] = value
- return self.load_data(table, UNIQUE_DATA_COLUMNS, kvfilter)
+ return self._load_data(table, UNIQUE_DATA_COLUMNS, kvfilter)
def save_unique_data(self, table, data):
q = None
@@ -272,7 +272,7 @@ class Store(Log):
except Exception, e: # pylint: disable=broad-except
self.error("Failed to delete data from %s: [%s]" % (table, e))
- def reset_data(self, table):
+ def _reset_data(self, table):
try:
q = self._query(self._db, table, UNIQUE_DATA_COLUMNS)
q.drop()
@@ -305,7 +305,7 @@ class AdminStore(Store):
def wipe_data(self, plugin):
table = plugin+"_data"
- self.reset_data(table)
+ self._reset_data(table)
class UserStore(Store):