From 1cac2103f0dc992e4f6d9a1edd0fd88d46780eea Mon Sep 17 00:00:00 2001 From: std-odoo Date: Thu, 28 May 2020 06:17:34 +0000 Subject: [PATCH] [FIX] mass_mailing_sale: correctly compute the `sale_invoiced_amount` Bug === If multiple invoice have different currency, the invoiced amount of the mailing will be wrongly computed. Task-2265628 closes odoo/odoo#58390 X-original-commit: d7193351343758bfae1567fd19f57b3a686bdf80 Signed-off-by: Thibault Delavallee (tde) --- addons/mass_mailing_sale/models/mailing_mailing.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/addons/mass_mailing_sale/models/mailing_mailing.py b/addons/mass_mailing_sale/models/mailing_mailing.py index f4a5ee71fac..6d8137d96f9 100644 --- a/addons/mass_mailing_sale/models/mailing_mailing.py +++ b/addons/mass_mailing_sale/models/mailing_mailing.py @@ -23,8 +23,8 @@ class MassMailing(models.Model): for mass_mailing in self: if self.user_has_groups('sales_team.group_sale_salesman') and self.user_has_groups('account.group_account_invoice'): domain = mass_mailing._get_sale_utm_domain() + [('state', 'not in', ['draft', 'cancel'])] - moves = self.env['account.move'].search_read(domain, ['amount_untaxed']) - mass_mailing.sale_invoiced_amount = sum(i['amount_untaxed'] for i in moves) + moves = self.env['account.move'].search_read(domain, ['amount_untaxed_signed']) + mass_mailing.sale_invoiced_amount = sum(i['amount_untaxed_signed'] for i in moves) else: mass_mailing.sale_invoiced_amount = 0