From fd7a6381d3e6c865341aa51f10e68e58df324818 Mon Sep 17 00:00:00 2001 From: Lorenzo Battistini Date: Wed, 25 Jul 2018 14:40:02 +0200 Subject: [PATCH] FIX l10n_it_reverse_charge RC payment move should be posted (#559) --- l10n_it_reverse_charge/__manifest__.py | 2 +- l10n_it_reverse_charge/models/account_invoice.py | 2 ++ l10n_it_reverse_charge/tests/test_rc.py | 6 ++++++ 3 files changed, 9 insertions(+), 1 deletion(-) diff --git a/l10n_it_reverse_charge/__manifest__.py b/l10n_it_reverse_charge/__manifest__.py index 3bd0103e6a4c..a184e008e07d 100644 --- a/l10n_it_reverse_charge/__manifest__.py +++ b/l10n_it_reverse_charge/__manifest__.py @@ -7,7 +7,7 @@ { 'name': 'Reverse Charge IVA', - 'version': '10.0.1.1.1', + 'version': '10.0.1.1.2', 'category': 'Localization/Italy', 'summary': 'Reverse Charge for Italy', 'author': 'Odoo Italia Network,Odoo Community Association (OCA)', diff --git a/l10n_it_reverse_charge/models/account_invoice.py b/l10n_it_reverse_charge/models/account_invoice.py index 85919aa94b3d..d669e626f24f 100644 --- a/l10n_it_reverse_charge/models/account_invoice.py +++ b/l10n_it_reverse_charge/models/account_invoice.py @@ -183,6 +183,7 @@ def reconcile_supplier_invoice(self): payment_debit_line = move_line elif move_line.credit: payment_credit_line = move_line + rc_payment.post() lines_to_rec = move_line_model.browse([ self.get_inv_line_to_reconcile().id, @@ -239,6 +240,7 @@ def reconcile_rc_invoice(self, rc_payment): (0, 0, rc_payment_debit_line_data), (0, 0, rc_payment_credit_line_data), ] + rc_payment.post() inv_line_to_reconcile = self.get_rc_inv_line_to_reconcile(rc_invoice) for move_line in rc_payment.line_ids: if move_line.account_id.id == inv_line_to_reconcile.account_id.id: diff --git a/l10n_it_reverse_charge/tests/test_rc.py b/l10n_it_reverse_charge/tests/test_rc.py index 012bc6094ee0..b6e1c0463bd2 100644 --- a/l10n_it_reverse_charge/tests/test_rc.py +++ b/l10n_it_reverse_charge/tests/test_rc.py @@ -203,6 +203,9 @@ def test_intra_EU(self): invoice.action_invoice_open() self.assertIsNot(bool(invoice.rc_self_invoice_id), False) self.assertEqual(invoice.rc_self_invoice_id.state, 'paid') + self.assertEqual( + invoice.rc_self_invoice_id.payment_move_line_ids.move_id.state, + 'posted') def test_extra_EU(self): supplier_extraEU = self.partner_model.create({ @@ -240,6 +243,9 @@ def test_extra_EU(self): invoice.action_invoice_open() self.assertIsNot(bool(invoice.rc_self_purchase_invoice_id), False) self.assertEqual(invoice.rc_self_purchase_invoice_id.state, 'paid') + self.assertEqual( + invoice.rc_self_purchase_invoice_id.payment_move_line_ids. + move_id.state, 'posted') def test_intra_EU_cancel_and_draft(self): supplier_intraEU = self.partner_model.create({