From 0e45c3a8ae8dfb1832a469c6daff970e32a89d33 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?C=C3=A9dric=20Krier?= <ced@b2ck.com>
Date: Sun, 17 Jan 2021 01:25:01 +0100
Subject: [PATCH] Allow product customer to be deactivated

issue8062
review333461002
---
 CHANGELOG                      |  2 ++
 product.py                     | 24 ++++++++++++++++--------
 view/product_customer_form.xml |  8 ++++++--
 3 files changed, 24 insertions(+), 10 deletions(-)

diff --git a/CHANGELOG b/CHANGELOG
index faae74b6bf..3c20c90096 100644
--- a/CHANGELOG
+++ b/CHANGELOG
@@ -1,3 +1,5 @@
+* Allow product customer to be deactivated
+
 Version 5.8.0 - 2020-11-02
 * Bug fixes (see mercurial logs for details)
 * Remove support for Python 3.5
diff --git a/product.py b/product.py
index d97fdddab4..465a84e0fa 100644
--- a/product.py
+++ b/product.py
@@ -1,14 +1,16 @@
 # This file is part of Tryton.  The COPYRIGHT file at the top level of
 # this repository contains the full copyright notices and license terms.
 
-from trytond.model import (fields, ModelSQL, ModelView, sequence_ordered,
-    MatchMixin)
+from trytond.model import (
+    ModelSQL, ModelView, sequence_ordered, MatchMixin, DeactivableMixin,
+    fields)
 from trytond.pool import PoolMeta, Pool
 from trytond.pyson import If, Eval, Bool
 from trytond.tools import lstrip_wildcard
 
 
-class ProductCustomer(sequence_ordered(), ModelSQL, ModelView, MatchMixin):
+class ProductCustomer(
+        sequence_ordered(), DeactivableMixin, ModelSQL, ModelView, MatchMixin):
     'Product Customer'
     __name__ = 'sale.product_customer'
 
@@ -18,17 +20,19 @@ class ProductCustomer(sequence_ordered(), ModelSQL, ModelView, MatchMixin):
         domain=[
             If(Bool(Eval('product')),
                 ('products', '=', Eval('product')),
-                ())
+                ()),
+            If(Eval('active'), ('active', '=', True), ()),
             ],
-        depends=['product'])
+        depends=['product', 'active'])
     product = fields.Many2One(
         'product.product', "Variant", select=True,
         domain=[
             If(Bool(Eval('template')),
                 ('template', '=', Eval('template')),
                 ()),
+            If(Eval('active'), ('active', '=', True), ()),
             ],
-        depends=['template'])
+        depends=['template', 'active'])
     party = fields.Many2One('party.party', "Customer", required=True,
         ondelete='CASCADE')
     name = fields.Char("Name", translate=True)
@@ -77,7 +81,10 @@ class Template(metaclass=PoolMeta):
         states={
             'invisible': ~Eval('salable', False),
             },
-        depends=['salable'])
+        domain=[
+            If(~Eval('active'), ('active', '=', False), ()),
+            ],
+        depends=['salable', 'active'])
 
     def product_customer_used(self, **pattern):
         for product_customer in self.product_customers:
@@ -116,11 +123,12 @@ class Product(metaclass=PoolMeta):
         'sale.product_customer', 'product', "Customers",
         domain=[
             ('template', '=', Eval('template')),
+            If(~Eval('active'), ('active', '=', False), ()),
             ],
         states={
             'invisible': ~Eval('salable', False),
             },
-        depends=['template', 'salable'])
+        depends=['template', 'salable', 'active'])
 
     def product_customer_used(self, **pattern):
         for product_customer in self.product_customers:
diff --git a/view/product_customer_form.xml b/view/product_customer_form.xml
index 5d64cc3b04..a5f844cced 100644
--- a/view/product_customer_form.xml
+++ b/view/product_customer_form.xml
@@ -8,8 +8,12 @@ this repository contains the full copyright notices and license terms. -->
     <field name="product" colspan="3"/>
     <label name="party"/>
     <field name="party"/>
-    <label name="sequence"/>
-    <field name="sequence"/>
+    <group id="sequence-active" col="-1" colspan="2">
+        <label name="sequence"/>
+        <field name="sequence"/>
+        <label name="active"/>
+        <field name="active"/>
+    </group>
     <label name="name"/>
     <field name="name"/>
     <label name="code"/>