From d163dba0c22f5f148d6e27b2b9406fcba6f97d21 Mon Sep 17 00:00:00 2001
From: Elouan Le Bars <elouan@coopiteasy.be>
Date: Thu, 6 Feb 2020 15:52:41 +0100
Subject: [PATCH] [REF] b_shift : attendance sheet validation wizard

---
 beesdoo_shift/wizard/validate_attendance_sheet.py  | 6 +++++-
 beesdoo_shift/wizard/validate_attendance_sheet.xml | 2 +-
 2 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/beesdoo_shift/wizard/validate_attendance_sheet.py b/beesdoo_shift/wizard/validate_attendance_sheet.py
index 2b295d3..dd90506 100644
--- a/beesdoo_shift/wizard/validate_attendance_sheet.py
+++ b/beesdoo_shift/wizard/validate_attendance_sheet.py
@@ -68,7 +68,7 @@ class ValidateAttendanceSheet(models.TransientModel):
     notes = fields.Text(related="active_sheet.notes", default="", readonly=False)
     feedback = fields.Text(related="active_sheet.feedback", default="", readonly=False)
     worker_nb_feedback = fields.Selection(
-        related="active_sheet.worker_nb_feedback", readonly=False, required=True
+        related="active_sheet.worker_nb_feedback", readonly=False
     )
 
     def on_barcode_scanned(self, barcode):
@@ -84,6 +84,10 @@ class ValidateAttendanceSheet(models.TransientModel):
     @api.multi
     def validate_sheet(self):
         sheet = self.active_sheet
+
+        if not self.worker_nb_feedback:
+            raise UserError(_("Please give your feedback on the number of workers."))
+
         if self.card_support:
             # Login with barcode
             card = self.env["member.card"].search(
diff --git a/beesdoo_shift/wizard/validate_attendance_sheet.xml b/beesdoo_shift/wizard/validate_attendance_sheet.xml
index 0d475a3..05de63b 100644
--- a/beesdoo_shift/wizard/validate_attendance_sheet.xml
+++ b/beesdoo_shift/wizard/validate_attendance_sheet.xml
@@ -35,7 +35,7 @@
                 <footer>
                     <button type="object"
                         name="validate_sheet"
-                        string="Validate"
+                        string="Comment and validate"
                         class="oe_highlight"
                     />
                     <button type="object" name="save"