diff --git a/addons/hr_recruitment/models/hr_recruitment.py b/addons/hr_recruitment/models/hr_recruitment.py index a7d2f2917bc..a685f7291d1 100644 --- a/addons/hr_recruitment/models/hr_recruitment.py +++ b/addons/hr_recruitment/models/hr_recruitment.py @@ -446,12 +446,12 @@ class Applicant(models.Model): view_id = self.env.ref('hr_recruitment.hr_applicant_view_form_interviewer').id return super().get_view(view_id, view_type, **options) - def _notify_compute_recipients(self, message, msg_vals): + def _notify_get_recipients(self, message, msg_vals): """ Do not notify members of the Recruitment Interviewer group, as this might leak some data they shouldn't have access to. """ - recipients = super()._notify_compute_recipients(message, msg_vals) + recipients = super()._notify_get_recipients(message, msg_vals) interviewer_group = self.env.ref('hr_recruitment.group_hr_recruitment_interviewer').id return [recipient for recipient in recipients if interviewer_group not in recipient['groups']] diff --git a/addons/hr_recruitment/tests/test_recruitment_interviewer.py b/addons/hr_recruitment/tests/test_recruitment_interviewer.py index aa8aff9a32d..ef451dabd6e 100644 --- a/addons/hr_recruitment/tests/test_recruitment_interviewer.py +++ b/addons/hr_recruitment/tests/test_recruitment_interviewer.py @@ -114,6 +114,8 @@ class TestRecruitmentInterviewer(MailCommon): 'interviewer_id': self.interviewer_user.id, }) + applicant.message_subscribe(partner_ids=[self.interviewer_user.partner_id.id]) + with self.mock_mail_gateway(): message = applicant.message_post(body='A super secret message', message_type='comment', subtype_xmlid='mail.mt_comment') @@ -121,7 +123,7 @@ class TestRecruitmentInterviewer(MailCommon): message.with_user(self.interviewer_user).read() try: - self._find_mail_mail_wpartners(self.interviewer_user.partner_id, False) + self._find_mail_mail_wpartners(self.interviewer_user.partner_id, None) except AssertionError: pass else: