From baf0d4acebf263e32ce53f54819cc63108cb720e Mon Sep 17 00:00:00 2001
From: Benjamin <benjamin@le-filament.com>
Date: Wed, 29 Apr 2020 18:13:52 +0200
Subject: [PATCH] [update] check staff function

---
 models/scop_partner_staff.py | 21 +++++++++------------
 1 file changed, 9 insertions(+), 12 deletions(-)

diff --git a/models/scop_partner_staff.py b/models/scop_partner_staff.py
index 75a21d7..641740d 100644
--- a/models/scop_partner_staff.py
+++ b/models/scop_partner_staff.py
@@ -62,22 +62,19 @@ class ScopPartnerStaff(models.Model):
     # ------------------------------------------------------
     # Contrains
     # ------------------------------------------------------
-    @api.constrains(
-        'staff_count',
-        'staff_shareholder_count'
-        'staff_average')
-    def _check_staff_count(self):
+    @api.one
+    @api.constrains('staff_count', 'staff_shareholder_count', 'staff_average')
+    def _check_staff(self):
         """
             Interdit de créer une ligne à 0 sauf lorsque
             cette ligne vient de la liste ministère
         """
-        for record in self:
-            if ((record.staff_count == 0
-                    or record.staff_shareholder_count == 0
-                    or record.staff_average == 0)
-                    and record.type_entry not in ('questionnaire', 'questionnaire_inscription')):
-                raise ValidationError(
-                    "Tous les effectifs doivent être renseignés et supérieurs à 0.")
+        if ((self.staff_count == 0
+                or self.staff_shareholder_count == 0
+                or self.staff_average == 0)
+                and self.type_entry not in ('questionnaire', 'questionnaire_inscription')):
+            raise ValidationError(
+                "Tous les effectifs doivent être renseignés et supérieurs à 0.")
 
     @api.constrains('effective_date')
     def _check_effective_date(self):
-- 
GitLab