From 66d7a1ffdf403364e150cc31d6f9d26ec2624f1b Mon Sep 17 00:00:00 2001
From: Carlos Dauden <carlos.dauden@tecnativa.com>
Date: Thu, 21 Apr 2022 15:58:52 +0200
Subject: [PATCH] [IMP] agreement_rebate: Improvements

---
 agreement_rebate/wizards/invoice_create_views.xml    | 4 +++-
 agreement_rebate/wizards/settlement_create.py        | 7 ++++---
 agreement_rebate/wizards/settlement_create_views.xml | 2 +-
 3 files changed, 8 insertions(+), 5 deletions(-)

diff --git a/agreement_rebate/wizards/invoice_create_views.xml b/agreement_rebate/wizards/invoice_create_views.xml
index 751faee8b..1f0b84e54 100644
--- a/agreement_rebate/wizards/invoice_create_views.xml
+++ b/agreement_rebate/wizards/invoice_create_views.xml
@@ -14,6 +14,9 @@
                     </group>
                 </group>
                 <group string="Invoice data">
+                    <group>
+                        <field name="domain" />
+                    </group>
                     <group>
                         <field name="invoice_type" />
                     </group>
@@ -30,7 +33,6 @@
                         <field name="invoice_group" />
                     </group>
                 </group>
-                <field name="domain" invisible="1" />
                 <group string="Agreement types">
                     <field
                         name="agreement_type_ids"
diff --git a/agreement_rebate/wizards/settlement_create.py b/agreement_rebate/wizards/settlement_create.py
index 629e77d77..323dca358 100644
--- a/agreement_rebate/wizards/settlement_create.py
+++ b/agreement_rebate/wizards/settlement_create.py
@@ -130,9 +130,10 @@ class AgreementSettlementCreateWiz(models.TransientModel):
     def _prepare_settlement_line(
         self, domain, group, agreement, line=False, section=False
     ):
-        amount = (
-            group[self._get_amount_field()] or 0.0 + agreement.additional_consumption
-        )
+        amount = group[self._get_amount_field()] or 0.0
+        if self.domain == "purchase":
+            amount = -amount
+        amount += agreement.additional_consumption
         amount_section = 0.0
         vals = {
             "agreement_id": agreement.id,
diff --git a/agreement_rebate/wizards/settlement_create_views.xml b/agreement_rebate/wizards/settlement_create_views.xml
index 4cd4c751d..7fcf59262 100644
--- a/agreement_rebate/wizards/settlement_create_views.xml
+++ b/agreement_rebate/wizards/settlement_create_views.xml
@@ -7,6 +7,7 @@
             <form string="Create settlements">
                 <group>
                     <field name="date" required="1" />
+                    <field name="domain" />
                 </group>
                 <group string="Dates filter">
                     <group>
@@ -20,7 +21,6 @@
                     </group>
                 </group>
                 <group string="Journals">
-                    <field name="domain" invisible="1" />
                     <field
                         name="journal_ids"
                         widget="many2many_tags"