From b90cd9c87686cbd24aa37fdacd014f94c7f85a07 Mon Sep 17 00:00:00 2001 From: "Pedro M. Baeza" Date: Wed, 8 Jan 2020 17:18:28 +0100 Subject: [PATCH] [IMP] contract: remove redundant method I have detected a method that was created as redundant and with the same technique used when preparing the line values, so better to have everything together in the same method instead of having it spread. --- contract_variable_quantity/__manifest__.py | 2 +- contract_variable_quantity/models/contract_line.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/contract_variable_quantity/__manifest__.py b/contract_variable_quantity/__manifest__.py index 885b4a77d..999749a4f 100644 --- a/contract_variable_quantity/__manifest__.py +++ b/contract_variable_quantity/__manifest__.py @@ -5,7 +5,7 @@ { 'name': 'Variable quantity in contract recurrent invoicing', - 'version': '12.0.2.1.1', + 'version': '12.0.2.1.2', 'category': 'Contract Management', 'license': 'AGPL-3', 'author': "Tecnativa," diff --git a/contract_variable_quantity/models/contract_line.py b/contract_variable_quantity/models/contract_line.py index 9494e7b48..714e8eb51 100644 --- a/contract_variable_quantity/models/contract_line.py +++ b/contract_variable_quantity/models/contract_line.py @@ -44,9 +44,9 @@ class AccountAnalyticInvoiceLine(models.Model): return quantity @api.multi - def _prepare_invoice_line(self, invoice_id=False): + def _prepare_invoice_line(self, invoice_id=False, invoice_values=False): vals = super(AccountAnalyticInvoiceLine, self)._prepare_invoice_line( - invoice_id=invoice_id + invoice_id=invoice_id, invoice_values=invoice_values, ) if ( 'quantity' in vals