checkbox-dev team mailing list archive
-
checkbox-dev team
-
Mailing list archive
-
Message #00067
[PATCH 03/11] plainbox:secure:config: use Variable.validate() in Variable.__set__()
Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@xxxxxxxxxxxxx>
---
plainbox/plainbox/impl/secure/config.py | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/plainbox/plainbox/impl/secure/config.py b/plainbox/plainbox/impl/secure/config.py
index 02ce661..9917157 100644
--- a/plainbox/plainbox/impl/secure/config.py
+++ b/plainbox/plainbox/impl/secure/config.py
@@ -231,10 +231,7 @@ class Variable(INameTracking):
:raises ValidationError: if the new value is incorrect
"""
# Check it against all validators
- for validator in self.validator_list:
- message = validator(self, new_value)
- if message is not None:
- raise ValidationError(self, new_value, message)
+ self.validate(new_value)
# Assign it to the backing store of the instance
instance._set_variable(self.name, new_value)
--
1.9.0
References