Change "var" for "attr" where it refer to an object attribute.

For example _persistent_vars is now _persistent_attrs. Mostly for
consistency and clarity.

diff --git a/services/dhcp/__init__.py b/services/dhcp/__init__.py
index 8bed88329314e42244f368b77aead6045d57e1ef..c9c25b49d7bbd9edd2ffbbae201bebd48e30ab15 100644 (file)
--- a/services/dhcp/__init__.py
+++ b/services/dhcp/__init__.py
@@ -149,7 +149,7 @@ class DhcpHandler(Restorable, ConfigWriter, InitdHandler, TransactionalHandler,
 
     _initd_name = 'dhcpd'
 
-    _persistent_vars = ('params', 'hosts')
+    _persistent_attrs = ('params', 'hosts')
 
     _restorable_defaults = dict(
             hosts = dict(),

diff --git a/services/dns/__init__.py b/services/dns/__init__.py
index ce21e97e7994fc40765484114e61f7bef7a5701a..2ff6f60b07296db83ee72d3e2138df7a91d94211 100644 (file)
--- a/services/dns/__init__.py
+++ b/services/dns/__init__.py
@@ -410,7 +410,7 @@ class DnsHandler(Restorable, ConfigWriter, InitdHandler, TransactionalHandler,
 
     _initd_name = 'bind'
 
-    _persistent_vars = ('params', 'zones')
+    _persistent_attrs = ('params', 'zones')
 
     _restorable_defaults = dict(
             zones = dict(),

diff --git a/services/firewall/__init__.py b/services/firewall/__init__.py
index 738bf67e82d7e79cfe44f147cb4322dd3d43130b..ca9bd5d866a6d958491efe8a3360de3cdf423f2f 100644 (file)
--- a/services/firewall/__init__.py
+++ b/services/firewall/__init__.py
@@ -186,7 +186,7 @@ class FirewallHandler(Restorable, ConfigWriter, ServiceHandler,
     Both defaults to the current working directory.
     """
 
-    _persistent_vars = 'rules'
+    _persistent_attrs = 'rules'
 
     _restorable_defaults = dict(rules=list())
 

diff --git a/services/ip/__init__.py b/services/ip/__init__.py
index cf16088fad959b8bb098432e762e202bd62ea73f..a27e0d805028ef01494d424c01dbcb046e684ea3 100644 (file)
--- a/services/ip/__init__.py
+++ b/services/ip/__init__.py
@@ -246,7 +246,7 @@ def get_devices():
 
 class IpHandler(Restorable, ConfigWriter, TransactionalHandler):
 
-    _persistent_vars = 'devices'
+    _persistent_attrs = 'devices'
 
     _restorable_defaults = dict(devices=get_devices())
 

diff --git a/services/proxy/__init__.py b/services/proxy/__init__.py
index 60548871886c13aac612a98783f2b60007da4283..6aff5cda686e48c7ed68da5275b5b861bfe9f7f7 100644 (file)
--- a/services/proxy/__init__.py
+++ b/services/proxy/__init__.py
@@ -123,7 +123,7 @@ class ProxyHandler(Restorable, ConfigWriter, InitdHandler,
 
     _initd_name = 'squid'
 
-    _persistent_vars = ('params', 'hosts')
+    _persistent_attrs = ('params', 'hosts')
 
     _restorable_defaults = dict(
             hosts = dict(),

diff --git a/services/util.py b/services/util.py
index 4468384c2152f234cd1dc4a91c1a5ccd6f92d419..7239867f34e1f8e1dbb12ad1ed7d43aa594b56cc 100644 (file)
--- a/services/util.py
+++ b/services/util.py
@@ -113,17 +113,17 @@ def call(command, stdin=subprocess.PIPE, stdout=subprocess.PIPE,
         raise ExecutionError(command, ReturnNot0Error(r))
 
 class Persistent:
-    r"""Persistent([vars[, dir[, ext]]]) -> Persistent.
+    r"""Persistent([attrs[, dir[, ext]]]) -> Persistent.
 
     This is a helper class to inherit from to automatically handle data
     persistence using pickle.
 
-    The variables attributes to persist (vars), and the pickle directory (dir)
+    The variables attributes to persist (attrs), and the pickle directory (dir)
     and file extension (ext) can be defined by calling the constructor or in a
     more declarative way as class attributes, like:
 
     class TestHandler(Persistent):
-        _persistent_vars = ('some_var', 'other_var')
+        _persistent_attrs = ('some_attr', 'other_attr')
         _persistent_dir = 'persistent-data'
         _persistent_ext = '.pickle'
 
@@ -138,14 +138,14 @@ class Persistent:
     # TODO implement it using metaclasses to add the handlers method by demand
     # (only for specifieds commands).
 
-    _persistent_vars = ()
+    _persistent_attrs = ()
     _persistent_dir = '.'
     _persistent_ext = '.pkl'
 
-    def __init__(self, vars=None, dir=None, ext=None):
+    def __init__(self, attrs=None, dir=None, ext=None):
         r"Initialize the object, see the class documentation for details."
-        if vars is not None:
-            self._persistent_vars = vars
+        if attrs is not None:
+            self._persistent_attrs = attrs
         if dir is not None:
             self._persistent_dir = dir
         if ext is not None:
@@ -153,28 +153,28 @@ class Persistent:
 
     def _dump(self):
         r"_dump() -> None :: Dump all persistent data to pickle files."
-        if isinstance(self._persistent_vars, basestring):
-            self._persistent_vars = (self._persistent_vars,)
-        for varname in self._persistent_vars:
-            self._dump_var(varname)
+        if isinstance(self._persistent_attrs, basestring):
+            self._persistent_attrs = (self._persistent_attrs,)
+        for attrname in self._persistent_attrs:
+            self._dump_attr(attrname)
 
     def _load(self):
         r"_load() -> None :: Load all persistent data from pickle files."
-        if isinstance(self._persistent_vars, basestring):
-            self._persistent_vars = (self._persistent_vars,)
-        for varname in self._persistent_vars:
-            self._load_var(varname)
-
-    def _dump_var(self, varname):
-        r"_dump_var() -> None :: Dump a especific variable to a pickle file."
-        f = file(self._pickle_filename(varname), 'wb')
-        pickle.dump(getattr(self, varname), f, 2)
+        if isinstance(self._persistent_attrs, basestring):
+            self._persistent_attrs = (self._persistent_attrs,)
+        for attrname in self._persistent_attrs:
+            self._load_attr(attrname)
+
+    def _dump_attr(self, attrname):
+        r"_dump_attr() -> None :: Dump a specific variable to a pickle file."
+        f = file(self._pickle_filename(attrname), 'wb')
+        pickle.dump(getattr(self, attrname), f, 2)
         f.close()
 
-    def _load_var(self, varname):
-        r"_load_var() -> object :: Load a especific pickle file."
-        f = file(self._pickle_filename(varname))
-        setattr(self, varname, pickle.load(f))
+    def _load_attr(self, attrname):
+        r"_load_attr() -> object :: Load a specific pickle file."
+        f = file(self._pickle_filename(attrname))
+        setattr(self, attrname, pickle.load(f))
         f.close()
 
     def _pickle_filename(self, name):
@@ -192,14 +192,14 @@ class Restorable(Persistent):
     declarative way as class attributes, like:
 
     class TestHandler(Restorable):
-        _persistent_vars = ('some_var', 'other_var')
+        _persistent_attrs = ('some_attr', 'other_attr')
         _restorable_defaults = dict(
-                some_var = 'some_default',
-                other_var = 'other_default')
+                some_attr = 'some_default',
+                other_attr = 'other_default')
 
-    The defaults is a dictionary, very coupled with the _persistent_vars
+    The defaults is a dictionary, very coupled with the _persistent_attrs
     attribute inherited from Persistent. The defaults keys should be the
-    values from _persistent_vars, and the values the default values.
+    values from _persistent_attrs, and the values the default values.
 
     The _restore() method returns True if the data was restored successfully
     or False if the defaults were loaded (in case you want to take further
@@ -569,7 +569,7 @@ if __name__ == '__main__':
     # Persistent test
     print 'PTestHandler'
     class PTestHandler(Persistent):
-        _persistent_vars = 'vars'
+        _persistent_attrs = 'vars'
         def __init__(self):
             self.vars = dict(a=1, b=2)
     h = PTestHandler()
@@ -591,7 +591,7 @@ if __name__ == '__main__':
     # Restorable test
     print 'RTestHandler'
     class RTestHandler(Restorable):
-        _persistent_vars = 'vars'
+        _persistent_attrs = 'vars'
         _restorable_defaults = dict(vars=dict(a=1, b=2))
         def __init__(self):
             self._restore()