From e417895d73445bf0901772663900783728cf0dcf Mon Sep 17 00:00:00 2001 From: houssine Date: Sat, 6 Apr 2019 14:11:11 +0200 Subject: [PATCH] [IMP] put iban control in a specific function to be reused --- easy_my_coop/models/coop.py | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/easy_my_coop/models/coop.py b/easy_my_coop/models/coop.py index 581040b..30a9007 100644 --- a/easy_my_coop/models/coop.py +++ b/easy_my_coop/models/coop.py @@ -103,16 +103,19 @@ class subscription_request(models.Model): return False return True + def check_iban(self, iban): + validated = True + try: + base_iban.validate_iban(iban) + except ValidationError: + validated = False + return validated + @api.multi @api.depends('iban', 'no_registre', 'skip_control_ng', 'is_company') def _validated_lines(self): for sub_request in self: - validated = False - try: - base_iban.validate_iban(sub_request.iban) - validated = True - except ValidationError: - validated = False + validated = self.check_iban(sub_request.iban) if validated and (sub_request.skip_control_ng or self.check_belgian_identification_id(