From 0d31a94e726b6292a35a1dfba2ebc6fd0bcfc82f Mon Sep 17 00:00:00 2001 From: Benjamin <benjamin@le-filament.com> Date: Wed, 6 May 2020 17:52:29 +0200 Subject: [PATCH] [clean] clean code --- models/res_partner.py | 29 ++++++++++++++--------------- 1 file changed, 14 insertions(+), 15 deletions(-) diff --git a/models/res_partner.py b/models/res_partner.py index 568b62e..e55e0aa 100644 --- a/models/res_partner.py +++ b/models/res_partner.py @@ -250,8 +250,8 @@ class ScopPartner(models.Model): 'res.partner', string='Organisme de révision suppléant', domain=[('active', '=', True), ('is_company', '=', True), - '|', ('organization_subtype_id.name', '=', - 'Organisme réviseur'), + '|', + ('organization_subtype_id.name', '=', 'Organisme réviseur'), ('organization_subtype_id.name', '=', 'Organisme réviseur, ' + 'Expert-comptable / Commissaire au compte')], on_delete='restrict') @@ -404,7 +404,7 @@ class ScopPartner(models.Model): if len(self.siret) != 14: raise ValidationError( "La longueur du SIRET doit être égale à 14") - if (self.search_count([('siret', '=like', siren+"%"), + if (self.search_count([('siret', '=like', siren + "%"), ('is_cooperative', "=", True)]) > 1): raise ValidationError( "Ce SIREN existe déjà parmi les coopératives") @@ -497,9 +497,9 @@ class ScopPartner(models.Model): @api.onchange('siret') def onchange_siret(self): if self.siret: - siren = (self.siret[:3] + " " + - self.siret[3:6] + " " + - self.siret[6:9]) + siren = (self.siret[:3] + " " + + self.siret[3:6] + " " + + self.siret[6:9]) if self.search_count([('siret', '=like', self.siret)]) > 0: return { 'warning': { @@ -507,7 +507,7 @@ class ScopPartner(models.Model): 'message': "Ce SIRET existe déjà, merci de vérifier " + "que vous n'êtes pas en train de créer un" + " doublon"}} - elif self.search_count([('siret', '=like', siren+"%")]) > 0: + elif self.search_count([('siret', '=like', siren + "%")]) > 0: return { 'warning': { 'title': "Attention", @@ -618,13 +618,13 @@ class ScopPartner(models.Model): def _compute_from_siret(self): for company in self: if company.siret: - company.siren = (company.siret[:3] + " " + - company.siret[3:6] + " " + - company.siret[6:9]) - company.formatted_siret = (company.siret[:3] + " " + - company.siret[3:6] + " " + - company.siret[6:9] + " " + - company.siret[9:]) + company.siren = (company.siret[:3] + " " + + company.siret[3:6] + " " + + company.siret[6:9]) + company.formatted_siret = (company.siret[:3] + " " + + company.siret[3:6] + " " + + company.siret[6:9] + " " + + company.siret[9:]) @api.depends('zip') def _compute_num_departement(self): @@ -787,7 +787,6 @@ class ScopPartner(models.Model): }) return True - @api.multi def scop_abandonne(self): self.write({ -- GitLab