[FIX] account_vat_period_end_statement: Select previous Statement by Account

Author: SirTakobi

account_vat_period_end_statement/models/account.py

@@ -1,6 +1,7 @@
 # Copyright 2011-2012 Domsense s.r.l. (<http://www.domsense.com>).
 # Copyright 2012-15 Agile Business Group sagl (<http://www.agilebg.com>)
 # Copyright 2015 Associazione Odoo Italia (<http://www.odoo-italia.org>)
+# Copyright 2022 Simone Rubino - TAKOBI
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
 import math
@@ -536,16 +537,28 @@ def create_move(self):
 
         return True
 
+    def _get_previous_statements(self):
+        self.ensure_one()
+        prev_statements = self.search(
+            [
+                ('date', '<', self.date),
+                ('annual', '=', False),
+            ],
+            order='date desc',
+        )
+        prev_statements_same_accounts = prev_statements.filtered(
+            lambda s: s.account_ids == self.account_ids
+        )
+        return prev_statements_same_accounts
+
     @api.multi
     def compute_amounts(self):
         decimal_precision_obj = self.env['decimal.precision']
         debit_line_model = self.env['statement.debit.account.line']
         credit_line_model = self.env['statement.credit.account.line']
         for statement in self:
             statement.previous_debit_vat_amount = 0.0
-            prev_statements = self.search(
-                [('date', '<', statement.date), ('annual', '=', False)],
-                order='date desc')
+            prev_statements = statement._get_previous_statements()
             if prev_statements and not statement.annual:
                 prev_statement = prev_statements[0]
                 if (

account_vat_period_end_statement/readme/CONTRIBUTORS.rst

@@ -9,4 +9,6 @@
 * Giacomo Grasso <[email protected]>
 * Lara Baggio <http://linkgroup.it/>
 * Gianmarco Conte <[email protected]>
+* `TAKOBI <https://takobi.online>`_:
 
+  * Simone Rubino <[email protected]>

account_vat_period_end_statement/tests/test_vat_statement.py

@@ -1,7 +1,9 @@
 #  Copyright 2015 Agile Business Group <http://www.agilebg.com>
+#  Copyright 2022 Simone Rubino - TAKOBI
 #  License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
-from odoo.tests.common import TransactionCase
+from odoo import fields
+from odoo.tests.common import TransactionCase, Form
 from datetime import datetime, date
 from dateutil.rrule import MONTHLY
 
@@ -227,3 +229,135 @@ def test_vat_statement(self):
         self.assertTrue(self.vat_statement.move_id)
         self.assertEqual(self.vat_statement.move_id.amount, 122)
         # TODO payment
+
+    def _create_vendor_bill(self, partner, date_invoice, price_unit, tax):
+        """
+        Create an open Vendor Bill for `partner` having date `date_invoice`.
+        The Bill will also have a Line having Price `price_unit` and Tax `tax`.
+        """
+        bill_model = self.invoice_model.with_context(
+            type='in_invoice',
+        )
+        bill_form = Form(bill_model)
+        bill_form.partner_id = partner
+        bill_form.date_invoice = date_invoice
+        with bill_form.invoice_line_ids.new() as line:
+            line.invoice_line_tax_ids.clear()
+            line.invoice_line_tax_ids.add(tax)
+            line.name = "Test Invoice Line"
+            line.price_unit = price_unit
+        bill = bill_form.save()
+        bill.action_invoice_open()
+        return bill
+
+    def _get_statement(self, period, statement_date, accounts):
+        """
+        Create a VAT Statement in date `statement_date`
+        for Period `period` and Accounts `accounts`.
+        """
+        # Create statement
+        statement_form = Form(self.vat_statement_model)
+        statement_form.journal_id = self.general_journal
+        statement_form.authority_vat_account_id = self.vat_authority
+        statement_form.payment_term_id = self.account_payment_term
+        statement_form.date = statement_date
+        statement_form.account_ids.clear()
+        for account in accounts:
+            statement_form.account_ids.add(account)
+        statement = statement_form.save()
+
+        # Add period
+        add_period_model = self.env['add.period.to.vat.statement']
+        add_period_model = add_period_model \
+            .with_context(
+                active_id=statement.id,
+                active_model=statement._name,
+            )
+        add_period_form = Form(add_period_model)
+        add_period_form.period_id = period
+        add_period = add_period_form.save()
+        add_period.add_period()
+        return statement
+
+    def test_different_previous_vat_statements(self):
+        """
+        Statements for different Accounts
+        only count previous Amounts from Statements with the same Accounts.
+        """
+        # Arrange: Create two different VAT Statements
+        # selecting two different Accounts
+        partner = self.env.ref('base.res_partner_4')
+        tax = self.account_tax_22_credit
+        tax_statement_account = tax.vat_statement_account_id
+        last_year_bill = self._create_vendor_bill(
+            partner, self.last_year_recent_date,
+            10, tax,
+        )
+        self.assertEqual(last_year_bill.state, 'open')
+        last_year_period = self.last_year_period
+        last_year_statement = self._get_statement(
+            last_year_period,
+            self.last_year_date,
+            tax_statement_account,
+        )
+        self.assertTrue(last_year_statement)
+
+        # Create another Bill and Statement
+        other_tax_statement_account = tax_statement_account.copy()
+        other_tax = tax.copy(
+            default={
+                'vat_statement_account_id': other_tax_statement_account.id,
+            },
+        )
+        other_last_year_bill = self._create_vendor_bill(
+            partner, self.last_year_recent_date,
+            20, other_tax,
+        )
+        self.assertEqual(other_last_year_bill.state, 'open')
+        last_year_period.type_id.allow_overlap = True
+        other_last_year_period = last_year_period.copy(
+            default={
+                'name': "Test Other last Year Period",
+                'vat_statement_id': False,
+            },
+        )
+        other_last_year_statement = self._get_statement(
+            other_last_year_period,
+            self.last_year_date,
+            other_tax_statement_account,
+        )
+        self.assertTrue(other_last_year_statement)
+
+        # Act: Create this Year's Statements,
+        # one for each Account used in previous Statements
+        today = fields.Date.today()
+        current_period = self.current_period
+        statement = self._get_statement(
+            current_period,
+            today,
+            tax_statement_account,
+        )
+
+        current_period.type_id.allow_overlap = True
+        other_current_period = current_period.copy(
+            default={
+                'name': "Test Other current Period",
+                'vat_statement_id': False,
+            },
+        )
+        other_statement = self._get_statement(
+            other_current_period,
+            today,
+            other_tax_statement_account,
+        )
+
+        # Assert: Each one of this Year's Statements counts as previous Amount
+        # only the corresponding last Year's Statement
+        self.assertEqual(
+            statement.previous_credit_vat_amount,
+            -last_year_statement.authority_vat_amount,
+        )
+        self.assertEqual(
+            other_statement.previous_credit_vat_amount,
+            -other_last_year_statement.authority_vat_amount,
+        )