diff --git a/addons/project_sms/models/project_project.py b/addons/project_sms/models/project_project.py index 27e21368b75..8c3de1bf602 100644 --- a/addons/project_sms/models/project_project.py +++ b/addons/project_sms/models/project_project.py @@ -13,7 +13,6 @@ class ProjectProject(models.Model): project._message_sms_with_template( template=project.stage_id.sms_template_id, partner_ids=project.partner_id.ids, - sms_numbers=project.partner_id.mapped('mobile'), ) @api.model_create_multi diff --git a/addons/project_sms/models/project_task.py b/addons/project_sms/models/project_task.py index d65bc4d9614..a40e69ba4e0 100644 --- a/addons/project_sms/models/project_task.py +++ b/addons/project_sms/models/project_task.py @@ -13,7 +13,6 @@ class ProjectTask(models.Model): task._message_sms_with_template( template=task.stage_id.sms_template_id, partner_ids=task.partner_id.ids, - sms_numbers=task.partner_id.mapped('mobile'), ) @api.model_create_multi diff --git a/addons/project_sms/tests/test_project_sharing.py b/addons/project_sms/tests/test_project_sharing.py index 45d456e937b..c53f125f524 100644 --- a/addons/project_sms/tests/test_project_sharing.py +++ b/addons/project_sms/tests/test_project_sharing.py @@ -44,7 +44,18 @@ class TestProjectSharingWithSms(TestProjectSharingCommon, SMSCommon): 'stage_id': self.task_stage_with_sms.id, }) self.assertEqual(self.task_portal.stage_id, self.task_stage_with_sms) - self.assertSMSIapSent([self.task_portal.partner_id.mobile]) + self.assertSMSIapSent([]) # no sms sent since the author is the recipient + + self.task_portal.write({ + 'partner_id': self.user_projectuser.partner_id.id, + 'stage_id': self.project_portal.type_ids[0].id, + }) + with self.mockSMSGateway(): + self.task_portal.with_user(self.user_portal).write({ + 'stage_id': self.task_stage_with_sms.id, + }) + self.assertEqual(self.task_portal.stage_id, self.task_stage_with_sms) + self.assertSMSIapSent([self.user_projectuser.partner_id.mobile]) with self.mockSMSGateway(): self.project_portal.write({