diff --git a/odex25_donation/ensan_sale_management/models/sale_order.py b/odex25_donation/ensan_sale_management/models/sale_order.py index 1463801f6..a9e7e51c5 100644 --- a/odex25_donation/ensan_sale_management/models/sale_order.py +++ b/odex25_donation/ensan_sale_management/models/sale_order.py @@ -86,24 +86,25 @@ class SaleOrder(models.Model): }) def action_confirm(self): - call_super = super().action_confirm() - if call_super == True: + res = super().action_confirm() + if res: + sms_template_id = self.env.ref('ensan_sale_management.sms_template_data_donation') + donar_sms_template_id = self.env.ref('ensan_sale_management.sms_template_donors_data_donation') for rec in self: - if rec.order_mobile_number: - sms_template_id = self.env.ref('ensan_sale_management.sms_template_data_donation') - rec._message_sms_with_template( - template=sms_template_id, - put_in_queue=False, - sms_numbers=[rec.order_mobile_number] + if rec.state == 'sale': + if rec.order_mobile_number: + rec._message_sms_with_template( + template=sms_template_id, + put_in_queue=False, + sms_numbers=[rec.order_mobile_number] + ) + for donator in rec.donators_ids: + donator._message_sms_with_template( + template=donar_sms_template_id, + put_in_queue=False, + sms_numbers=[donator.donator_mobile_number] ) - donar_sms_template_id = self.env.ref('ensan_sale_management.sms_template_donors_data_donation') - for donator in rec.donators_ids: - donator._message_sms_with_template( - template=donar_sms_template_id, - put_in_queue=False, - sms_numbers=[donator.donator_mobile_number] - ) - return call_super + return res @api.depends('donators_ids') def _compute_receiver_details(self):