From 57875ff2994095040364b8545f3d317a92b45768 Mon Sep 17 00:00:00 2001 From: Elouan Le Bars Date: Mon, 13 Jan 2020 14:40:32 +0100 Subject: [PATCH] [CHG] b_shift : attendance sheets terminology --- beesdoo_shift/data/mail_template.xml | 5 +- beesdoo_shift/models/attendance_sheet.py | 47 ++++++++++++------- beesdoo_shift/tests/test_attendance_sheet.py | 2 +- beesdoo_shift/views/attendance_sheet.xml | 16 ++++--- .../wizard/validate_attendance_sheet.py | 20 ++++---- .../wizard/validate_attendance_sheet.xml | 2 +- 6 files changed, 54 insertions(+), 38 deletions(-) diff --git a/beesdoo_shift/data/mail_template.xml b/beesdoo_shift/data/mail_template.xml index db27595..eb72e0c 100644 --- a/beesdoo_shift/data/mail_template.xml +++ b/beesdoo_shift/data/mail_template.xml @@ -38,10 +38,11 @@ % endif

Your status have been updated to "${object.worker_id.state}". -

If you have any question regarding this non-attendance, feel free to contact us. +

If you have any question regarding this non-attendance, just answer this e-mail.


-

Sustainably yours,

+

Cooperatively yours,
+ The Members' office volunteers

${object.worker_id.company_id.name}.

% if object.worker_id.company_id.street: diff --git a/beesdoo_shift/models/attendance_sheet.py b/beesdoo_shift/models/attendance_sheet.py index c7b000c..bad8a1c 100644 --- a/beesdoo_shift/models/attendance_sheet.py +++ b/beesdoo_shift/models/attendance_sheet.py @@ -181,27 +181,27 @@ class AttendanceSheet(models.Model): readonly=True, help="Indicative maximum number of workers.", ) - annotation = fields.Text("Annotation", default="") + notes = fields.Text("Notes", default="") is_annotated = fields.Boolean( compute="_compute_is_annotated", - string="Annotation", + string="Is annotated", readonly=True, store=True, ) is_read = fields.Boolean( string="Mark as read", - help="Has annotation been read by an administrator ?", + help="Has notes been read by an administrator ?", default=False, track_visibility="onchange", ) - feedback = fields.Text("Feedback") + feedback = fields.Text("Comments about the shift") worker_nb_feedback = fields.Selection( [ - ("not_enough", "Not enough"), - ("enough", "Enough"), - ("too_many", "Too many"), + ("not_enough", "Not enough workers"), + ("enough", "Enough workers"), + ("too_many", "Too many workers"), ], - string="Feedback on number of workers", + string="Was your team big enough ?", ) validated_by = fields.Many2one( "res.partner", @@ -218,7 +218,7 @@ class AttendanceSheet(models.Model): _sql_constraints = [ ( - "check_no_annotation_mark_read", + "check_not_annotated_mark_as_read", "CHECK ((is_annotated=FALSE AND is_read=FALSE) OR is_annotated=TRUE)", _("Non-annotated sheets can't be marked as read."), ) @@ -284,11 +284,11 @@ class AttendanceSheet(models.Model): if shift.task_id.planning_id.name: rec.week = shift.task_id.planning_id.name - @api.depends("annotation") + @api.depends("notes") def _compute_is_annotated(self): for rec in self: - if rec.annotation: - return bool(rec.annotation.strip()) + if rec.notes: + return bool(rec.notes.strip()) return False @api.constrains("expected_shift_ids", "added_shift_ids") @@ -312,7 +312,7 @@ class AttendanceSheet(models.Model): @api.constrains( "expected_shift_ids", "added_shift_ids", - "annotation", + "notes", "feedback", "worker_nb_feedback", ) @@ -336,14 +336,26 @@ class AttendanceSheet(models.Model): _("Multiple workers are corresponding this barcode.") ) - if worker.state in ("unsubscribed", "resigning"): - raise UserError(_("Worker is %s.") % worker.state) + if worker.state == "unsubscribed": + shift_counter = worker.cooperative_status_ids.sc + worker.cooperative_status_ids.sr + raise UserError(_( + "Beware, your account is frozen because your shift counter " + "is at %s. Please contact Members Office to unfreeze it. " + "If you want to attend this shift, your supercoop " + "can write your name in the notes field during validation." + ) % shift_counter) + if worker.state == "resigning": + raise UserError(_( + "Beware, you are recorded as resigning. " + "Please contact member's office if this is incorrect. Thank you." + )) if worker.working_mode not in ("regular", "irregular"): raise UserError( - _("Worker is %s and should be regular or irregular.") - % worker.working_mode + _("%s is %s and should be regular or irregular.") + % worker.name, worker.working_mode ) + # expected shifts status update for id in self.expected_shift_ids.ids: shift = self.env["beesdoo.shift.sheet.expected"].browse(id) if ( @@ -363,6 +375,7 @@ class AttendanceSheet(models.Model): if worker.id in added_ids: return + # added shift creation self.added_shift_ids |= self.added_shift_ids.new( { "task_type_id": self.added_shift_ids.default_task_type_id(), diff --git a/beesdoo_shift/tests/test_attendance_sheet.py b/beesdoo_shift/tests/test_attendance_sheet.py index 9ef03be..ecd8c75 100644 --- a/beesdoo_shift/tests/test_attendance_sheet.py +++ b/beesdoo_shift/tests/test_attendance_sheet.py @@ -247,7 +247,7 @@ class TestAttendanceSheet(TransactionCase): self.assertEquals(sheet_1.day_abbrevation, "Lundi") self.assertEquals(sheet_1.week, "Semaine A") self.assertTrue(sheet_1.name) - self.assertFalse(sheet_1.annotation) + self.assertFalse(sheet_1.notes) self.assertFalse(sheet_1.is_annotated) def test_attendance_sheet_barcode_scan(self): diff --git a/beesdoo_shift/views/attendance_sheet.xml b/beesdoo_shift/views/attendance_sheet.xml index bf21c84..becea88 100644 --- a/beesdoo_shift/views/attendance_sheet.xml +++ b/beesdoo_shift/views/attendance_sheet.xml @@ -140,8 +140,9 @@ name="validate_with_checks" string="Validate Sheet" confirm="Beware : a validated sheet cannot be edited anymore - and you won't be able to add any latecomers. - Emails will be sent to workers who did not attend their shift." + and you won't be able to add any latecomers. + The counters of those who didn't attend will be updated + and they will get warning emails." attrs="{'invisible': [('state', '=', 'validated')]}" /> @@ -170,8 +171,9 @@ -