From 73b004d071464603cd60252d70fa03f4cb8b9aff Mon Sep 17 00:00:00 2001 From: Romain Bignon Date: Wed, 12 Feb 2020 22:33:58 +0100 Subject: [PATCH] ValueTransient: change way to bypass check, and set default value to None --- weboob/tools/value.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/weboob/tools/value.py b/weboob/tools/value.py index 5c479b7803..0b7b07d675 100644 --- a/weboob/tools/value.py +++ b/weboob/tools/value.py @@ -100,10 +100,10 @@ def check_valid(self, v): :raises: ValueError """ - if self.default is not None and v == self.default: - return if self.required and v is None: raise ValueError('Value is required and thus must be set') + if v == self.default: + return if v == '' and self.default != '' and (self.choices is None or v not in self.choices): raise ValueError('Value can\'t be empty') if self.regexp is not None and not re.match(self.regexp + '$', unicode(v) if v is not None else ''): @@ -152,7 +152,7 @@ def get(self): class ValueTransient(Value): def __init__(self, *args, **kwargs): kwargs.setdefault('transient', True) - kwargs.setdefault('default', '') + kwargs.setdefault('default', None) kwargs.setdefault('required', False) super(ValueTransient, self).__init__(*args, **kwargs) -- GitLab