Fix bug in parameter setting (was not using __setattr__)

diff --git a/tests/test_tools_parameters.py b/tests/test_tools_parameters.py
index 145de7bcdc17610394d604c7d20338a237217367..539d4a811a76c1ad2315989f5e2c327f8ad443fa 100644 (file)
--- a/tests/test_tools_parameters.py
+++ b/tests/test_tools_parameters.py
@@ -39,11 +39,15 @@ class TestHasParam(unittest.TestCase):
         param_json = self.has_param_cls.get_parameters()
         param_dict = json.loads(param_json)
         self.assertEqual(param_dict, new_param_dict)
+        for name, value in new_param_dict.items():
+            self.assertEqual(self.has_param_cls.__getattribute__(name), value)
         # set from JSON
         self.has_param_cls.set_parameters(new_param_json)
         param_json = self.has_param_cls.get_parameters()
         param_dict = json.loads(param_json)
         self.assertEqual(param_dict, new_param_dict)
+        for name, value in new_param_dict.items():
+            self.assertEqual(self.has_param_cls.__getattribute__(name), value)
 
     def test_param_view(self):
         "param_view method"

diff --git a/timeside/tools/parameters.py b/timeside/tools/parameters.py
index 75f43ae5f210b853d680c1899948276cc4711071..871ab7be473a97249ec5f07a67a14be60392fa68 100644 (file)
--- a/timeside/tools/parameters.py
+++ b/timeside/tools/parameters.py
@@ -87,7 +87,8 @@ class HasParam(object):
         if isinstance(parameters, basestring):
             self.set_parameters(json.loads(parameters))
         else:
-            self._parameters.set(**parameters)
+            for name, value in parameters.items():
+                self.__setattr__(name, value)
 
     def validate_parameters(self, parameters):
         """Validate parameters format against Traits specification