diff options
| author | std-odoo <std@odoo.com> | 2020-05-28 06:17:34 +0000 |
|---|---|---|
| committer | std-odoo <std@odoo.com> | 2020-09-24 08:46:10 +0000 |
| commit | b22a8e509c8760bbda1afd10d823f569ed1dcacd (patch) | |
| tree | d93f8db930298986af7c146d83678825e88b35a4 | |
| parent | 812280896120a9950363c2cd775a49d2ac3a8695 (diff) | |
[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#58389
X-original-commit: d7193351343758bfae1567fd19f57b3a686bdf80
Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
| -rw-r--r-- | addons/mass_mailing_sale/models/mailing_mailing.py | 4 |
1 files 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 |