diff options
| author | Swapnesh Shah <swapneshshah111@gmail.com> | 2020-02-15 09:41:53 +0000 |
|---|---|---|
| committer | Swapnesh Shah <swapneshshah111@gmail.com> | 2020-04-03 18:41:06 +0000 |
| commit | cbd103eafc2333435167c0d747f057ec5177bf21 (patch) | |
| tree | a055d4f9665030019fde8ac99cdc7a528c1f88f2 /addons | |
| parent | 9d37cecfca8e6b8d4c0e2880f422673c526a3aef (diff) | |
[FIX] sale_coupon: expected singleton archiving coupon
Before this commit, There was expected singleton tracback on Archiving multiple Coupons at once.
With this commit, We are passing single Record to process.
This is manual Forward port of https://github.com/odoo/enterprise/pull/9637
(Coupons modules went from OE to OC in saas-12.3)
closes odoo/odoo#48996
X-original-commit: d8c2c7269560920511598414741ad42e1f7a0213
Signed-off-by: Romain Derie <rdeodoo@users.noreply.github.com>
Diffstat (limited to 'addons')
| -rw-r--r-- | addons/sale_coupon/models/sale_coupon_program.py | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/addons/sale_coupon/models/sale_coupon_program.py b/addons/sale_coupon/models/sale_coupon_program.py index d7db3360316..a7798302444 100644 --- a/addons/sale_coupon/models/sale_coupon_program.py +++ b/addons/sale_coupon/models/sale_coupon_program.py @@ -119,10 +119,10 @@ class SaleCouponProgram(models.Model): def toggle_active(self): super(SaleCouponProgram, self).toggle_active() - if not self.active: - coupons = self.filtered(lambda p: p.promo_code_usage == 'code_needed').mapped('coupon_ids') - coupons.filtered(lambda x: x.state != 'used').write({'state': 'expired'}) - self.mapped('discount_line_product_id').write({'active': self.active}) + for program in self: + program.discount_line_product_id.active = program.active + coupons = self.filtered(lambda p: not p.active and p.promo_code_usage == 'code_needed').mapped('coupon_ids') + coupons.filtered(lambda x: x.state != 'used').write({'state': 'expired'}) def action_view_sales_orders(self): self.ensure_one() |