From 9dabd9dd44d59a23747467ff29ad8ecb7bccba31 Mon Sep 17 00:00:00 2001
From: Andrey Solodovnikov <dron1611@mail.ru>
Date: Wed, 30 Aug 2023 09:32:07 +0300
Subject: [PATCH 1/2] [FIX] sale_coupon_reward_fixed_price:fix double
 percentage view

---
 sale_coupon_reward_fixed_price/views/coupon_program_view.xml | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sale_coupon_reward_fixed_price/views/coupon_program_view.xml b/sale_coupon_reward_fixed_price/views/coupon_program_view.xml
index 4078b98ae..2214d257c 100644
--- a/sale_coupon_reward_fixed_price/views/coupon_program_view.xml
+++ b/sale_coupon_reward_fixed_price/views/coupon_program_view.xml
@@ -38,12 +38,12 @@
             <xpath expr="//label[@for='discount_type']" position="attributes">
                 <attribute
                     name="attrs"
-                >{"invisible": [("reward_type", "in", ("product", "free_shipping", "fixed_price"))]}</attribute>
+                >{"invisible": [("reward_type", "in", ("product", "free_shipping", "fixed_price", "discount_line"))]}</attribute>
             </xpath>
             <xpath expr="//field[@name='discount_type']/.." position="attributes">
                 <attribute
                     name="attrs"
-                >{"invisible": [("reward_type", "in", ("product", "free_shipping", "fixed_price"))]}</attribute>
+                >{"invisible": [("reward_type", "in", ("product", "free_shipping", "fixed_price", "discount_line"))]}</attribute>
             </xpath>
             <xpath expr="//label[@for='discount_max_amount']" position="attributes">
                 <attribute

From 48b18b4cda4e0b0863373fb517834147e4390e79 Mon Sep 17 00:00:00 2001
From: Andrey Solodovnikov <dron1611@mail.ru>
Date: Wed, 30 Aug 2023 09:36:58 +0300
Subject: [PATCH 2/2] [FIX] sale_promotion_discount_in_field: Fix view of
 fields "Free product" and "apply discount" and "max discount amount"

This field are visible while they are not applicable to fixed price module
---
 .../views/coupon_program_view.xml                 | 15 +++++----------
 1 file changed, 5 insertions(+), 10 deletions(-)

diff --git a/sale_promotion_discount_in_field/views/coupon_program_view.xml b/sale_promotion_discount_in_field/views/coupon_program_view.xml
index a3c6b5e30..dc71d056b 100644
--- a/sale_promotion_discount_in_field/views/coupon_program_view.xml
+++ b/sale_promotion_discount_in_field/views/coupon_program_view.xml
@@ -7,8 +7,7 @@
         <field name="arch" type="xml">
             <field name="reward_product_id" position="attributes">
                 <attribute name="attrs">
-                    {"invisible": [("reward_type", "in", ("discount", "free_shipping",
-                    "discount_line"))], "required": [("reward_type", "=", "product")]}
+                    {"invisible": [("reward_type", "!=", "product")], "required": [("reward_type", "=", "product")]}
                 </attribute>
             </field>
             <field name="discount_line_product_id" position="attributes">
@@ -19,14 +18,12 @@
             </field>
             <xpath expr="//field[@name='discount_type']/.." position="attributes">
                 <attribute name="attrs">
-                    {"invisible": [("reward_type", "in", ("product", "free_shipping",
-                    "discount_line"))]}
+                    {"invisible": [("reward_type", "!=", "discount")]}
                 </attribute>
             </xpath>
             <xpath expr="//label[@for='discount_type']" position="attributes">
                 <attribute name="attrs">
-                    {"invisible": [("reward_type", "in", ("product", "free_shipping",
-                    "discount_line"))]}
+                    {"invisible": [("reward_type", "!=", "discount")]}
                 </attribute>
             </xpath>
             <xpath expr="//label[@for='discount_type']" position="before">
@@ -49,15 +46,13 @@
             </xpath>
             <xpath expr="//label[@for='discount_max_amount']" position="attributes">
                 <attribute name="attrs">
-                    {'invisible': ['|', ('reward_type', 'in', ('product',
-                    'free_shipping', "discount_line")), ('discount_type', '!=',
+                    {'invisible': ['|', ('reward_type', '!=', 'discount'), ('discount_type', '!=',
                     'percentage')]}
                 </attribute>
             </xpath>
             <xpath expr="//field[@name='discount_max_amount']/.." position="attributes">
                 <attribute name="attrs">
-                    {'invisible': ['|', ('reward_type', 'in', ('product',
-                    'free_shipping', "discount_line")), ('discount_type', '!=',
+                    {'invisible': ['|', ('reward_type', '!=', 'discount'), ('discount_type', '!=',
                     'percentage')]}
                 </attribute>
             </xpath>