From 4199b5d3f1787d82ccbda9190b9b3011e45ef9e1 Mon Sep 17 00:00:00 2001
From: Haresh Chavda <haresh@techreceptives.com>
Date: Thu, 14 Jun 2018 11:51:27 +0530
Subject: [PATCH] [IMP] Give reference of discount method in Credit Notes and
 add blank div in customer invoice form

---
 addons/account/views/account_invoice_view.xml |  2 ++
 .../models/account_invoice.py                 | 28 +++++++++++++++----
 .../views/account_invoice_views.xml           |  6 ++--
 3 files changed, 27 insertions(+), 9 deletions(-)

diff --git a/addons/account/views/account_invoice_view.xml b/addons/account/views/account_invoice_view.xml
index ffeaf4e7..c9521e24 100644
--- a/addons/account/views/account_invoice_view.xml
+++ b/addons/account/views/account_invoice_view.xml
@@ -226,6 +226,8 @@
                 </div>
                 <field name="has_outstanding" invisible="1"/>
                 <sheet string="Vendor Bill">
+                    <div class="oe_button_box" name="button_box">
+                    </div>
                     <div>
                         <label string="Draft Bill" attrs="{'invisible': ['|',('state','!=','draft'), ('type','!=','in_invoice')]}"/>
                         <label string="- First Number:" attrs="{'invisible': [('sequence_number_next_prefix','=',False)]}"/>
diff --git a/addons/account_discount/models/account_invoice.py b/addons/account_discount/models/account_invoice.py
index 276de0f2..16ad498b 100644
--- a/addons/account_discount/models/account_invoice.py
+++ b/addons/account_discount/models/account_invoice.py
@@ -9,16 +9,17 @@ class AccountInvoice(models.Model):
     _inherit = "account.invoice"
 
     @api.multi
-    @api.depends('discount_amount', 'discount_per', 'amount_untaxed')
+    @api.depends('discount_amount', 'discount_per', 'amount_untaxed',
+                 'invoice_line_ids')
     def _get_discount(self):
-        total_discount = 0.0
         for record in self:
+            total_discount = 0.0
             for invoice_line_id in record.invoice_line_ids:
                 total_price = (
                     invoice_line_id.quantity * invoice_line_id.price_unit)
                 total_discount += \
                     (total_price * invoice_line_id.discount) / 100
-        record.discount = record.currency_id.round(total_discount)
+            record.discount = record.currency_id.round(total_discount)
 
     @api.multi
     @api.depends('invoice_line_ids', 'discount_per', 'discount_amount')
@@ -40,7 +41,6 @@ class AccountInvoice(models.Model):
 
     @api.multi
     def calculate_discount(self):
-        self._check_constrains()
         for line in self.invoice_line_ids:
             line.write({'discount': 0.0})
         # amount_untaxed = self.amount_untaxed
@@ -58,6 +58,21 @@ class AccountInvoice(models.Model):
                     (discount_value_ratio * 100) / line.price_subtotal
                 line.write({'discount': discount_per_ratio})
                 self._onchange_invoice_line_ids()
+        self._check_constrains()
+
+    @api.multi
+    @api.returns('self')
+    def refund(self, date_invoice=None,
+               date=None, description=None, journal_id=None):
+        result = super(AccountInvoice, self).refund(
+            date_invoice=date_invoice, date=date,
+            description=description, journal_id=journal_id)
+        result.write({
+            'discount_method': result.refund_invoice_id.discount_method,
+            'discount_amount': result.refund_invoice_id.discount_amount,
+            'discount_per': result.refund_invoice_id.discount_per})
+        result.calculate_discount()
+        return result
 
     @api.constrains('discount_per', 'discount_amount', 'invoice_line_ids')
     def _check_constrains(self):
@@ -66,8 +81,9 @@ class AccountInvoice(models.Model):
 
     @api.onchange('discount_method')
     def onchange_discount_method(self):
-        self.discount_amount = 0.0
-        self.discount_per = 0.0
+        if not self.refund_invoice_id:
+            self.discount_amount = 0.0
+            self.discount_per = 0.0
         if self.discount_method and not self.invoice_line_ids:
             raise Warning('No Invoice Line(s) were found!')
 
diff --git a/addons/account_discount/views/account_invoice_views.xml b/addons/account_discount/views/account_invoice_views.xml
index a008519c..cd350983 100644
--- a/addons/account_discount/views/account_invoice_views.xml
+++ b/addons/account_discount/views/account_invoice_views.xml
@@ -9,9 +9,9 @@
         <field name="arch" type="xml">
             <xpath expr="//group[@name='left_panel']" position="inside">
                 <separator string="Discount"/>
-                <field name="discount_method" attrs="{'readonly': ['|', ('origin', '!=', False), ('state', '!=', 'draft')]}"/>
-                <field name="discount_amount" attrs="{'readonly': ['|', '|', ('origin', '!=', False), ('state', '!=', 'draft'), ('discount_method', '=', False)], 'invisible': [('discount_method', '=', 'per')]}"/>
-                <field name="discount_per" attrs="{'invisible': ['|', ('discount_method', '=', 'fixed'), ('discount_method', '=', False)], 'readonly': ['|', ('origin', '!=', False), ('state', '!=', 'draft')]}"/>
+                <field name="discount_method" attrs="{'readonly': [('state', '!=', 'draft')]}"/>
+                <field name="discount_amount" attrs="{'readonly': [('state', '!=', 'draft'), ('discount_method', '=', False)], 'invisible': [('discount_method', '=', 'per')]}"/>
+                <field name="discount_per" attrs="{'invisible': ['|', ('discount_method', '=', 'fixed'), ('discount_method', '=', False)], 'readonly': [('state', '!=', 'draft')]}"/>
                 <button name="calculate_discount" type="object" class="oe_edit_only oe_link" string="(Update Discount)" attrs="{'invisible': [('discount_method', '=', False)]}" confirm="If you want to apply global discount other discount methods will be reset."/>
             </xpath>
             <xpath expr="//field[@name='amount_untaxed']" position="before">