mirror of
https://github.com/OCA/account-financial-tools.git
synced 2025-02-02 12:47:26 +02:00
[ADD] account_cash_basis_group_base_line: add module
account: Compacting the creation of Journal Lines for CABA base lines. Main - Creation of Cash Basis (CABA) Journal Entries can lead to the creation of a huge number of lines that are not quite useful and its creations can lead to a lengthy process of creation of payment when several invoices with a huge amount of invoice lines involved or to a lengthy process of cancelation of payment because of reversion of the CABA, in case of l10n_mx, many users have opted for using l10n_mx_edi_avoid_reversal_entry, and then this causes a lengthy process of deletions of CABA entries. Justification - CABA Entry is divided into two parts when it is created - Tax Part: The part that deals with the taxes themselves (this PR does not mess with them). - Base Part: The part of the base of the taxes (what this PR is all about) So far the Base Part has been using a Brigde/Order account that does not have any financial use other than providing the source for computing the base of the amount of taxes due. In the financial statements, the value of the account used is always zero. So providing a huge amount of details on this account has proved, in the wild, of no use. Scenario - - Configure Tax based on payment. - Configure the tax base account in the Tax. - Have four invoices, each one with one hundred invoice lines, each line is a different product or a different account. (4 invoices x 100 lines = 400 lines to be used as a tax base) - Make three partial payments to each invoice. (3 payments x 4 invoices = 12 payments) Before this change - - Check the CABA entry created: Chances are that each CABA Journal Entry will end up having: -- 1 Entry line per tax collected. -- 1 Entry line per tax to be collected. -- 200 Entry lines for the base (100 lines in debit, 100 lines in credit) That is a **whopping 2424 lines** for CABA entries where 2400 are only base lines. And let us say that I have made a mistake on all the 12 payment (I am that dumb). Reversal Method will clone your 2424 lines so now there are 2424 additional lines. And remember I have to fix my payments and that creates again 2424 additional lines. So **I end up with 2424 x 3 = 7272 lines.** This only for 3 invoices and 4 payment, for payments canceled and their corrections. After this change - - Check the CABA entry created: -- 1 Entry line per tax collected. -- 1 Entry line per tax to be collected. -- 2 Entry lines for the base, one debit, one credit. Chances are that this could be increased but the algorithm tries only to group by a set of keys. So with the same steps, payments reversals and corrections, I could end up with 12 payment x 4 lines x 3 steps = **144 lines down from 7272 lines.**
This commit is contained in:
committed by
Gabriela Mogollón
parent
5c12286103
commit
45e9176fd1
0
account_cash_basis_group_base_line/README.rst
Normal file
0
account_cash_basis_group_base_line/README.rst
Normal file
1
account_cash_basis_group_base_line/__init__.py
Normal file
1
account_cash_basis_group_base_line/__init__.py
Normal file
@@ -0,0 +1 @@
|
||||
from . import models
|
||||
13
account_cash_basis_group_base_line/__manifest__.py
Normal file
13
account_cash_basis_group_base_line/__manifest__.py
Normal file
@@ -0,0 +1,13 @@
|
||||
{
|
||||
'name': 'Tax Cash Basis Group Base Lines',
|
||||
'version': '12.0.0.0.1',
|
||||
'summary': 'Compacting the creation of Journal Lines for CABA base lines.',
|
||||
'sequence': 5,
|
||||
'category': 'Accounting',
|
||||
'depends': ['account'],
|
||||
'author': 'Vauxoo, Odoo Community Association (OCA)',
|
||||
'license': 'AGPL-3',
|
||||
'data': [],
|
||||
'installable': True,
|
||||
'auto_install': False,
|
||||
}
|
||||
1
account_cash_basis_group_base_line/models/__init__.py
Normal file
1
account_cash_basis_group_base_line/models/__init__.py
Normal file
@@ -0,0 +1 @@
|
||||
from . import account_move
|
||||
22
account_cash_basis_group_base_line/models/account_move.py
Normal file
22
account_cash_basis_group_base_line/models/account_move.py
Normal file
@@ -0,0 +1,22 @@
|
||||
from odoo import models
|
||||
|
||||
|
||||
class AccountPartialReconcile(models.Model):
|
||||
_inherit = "account.partial.reconcile"
|
||||
|
||||
def _get_tax_cash_basis_base_key(self, tax, move, line):
|
||||
account_id = self._get_tax_cash_basis_base_account(line, tax)
|
||||
return (move.id, account_id.id, tax.id, line.currency_id.id, line.partner_id.id)
|
||||
|
||||
def _get_tax_cash_basis_base_common_vals(self, key, new_move):
|
||||
move, account_id, tax, currency_id, partner_id = key
|
||||
move = self.env['account.move'].browse(move)
|
||||
return {
|
||||
'name': move.name,
|
||||
'account_id': account_id,
|
||||
'tax_exigible': True,
|
||||
'tax_ids': [(6, 0, [tax])],
|
||||
'move_id': new_move.id,
|
||||
'currency_id': currency_id,
|
||||
'partner_id': partner_id,
|
||||
}
|
||||
@@ -0,0 +1,2 @@
|
||||
* Humberto Arocha <hbto@vauxoo.com>
|
||||
* Gabriela Mogollón <gmogollon@vauxoo.com>
|
||||
@@ -0,0 +1,4 @@
|
||||
This module compact the creation of Journal Lines for Cash Basis base lines.
|
||||
|
||||
Instead of creating two base lines per line with cash basis tax, let us group
|
||||
the creation lines so that there are fewer lines in the Cash Basis Journal Entry.
|
||||
1
account_cash_basis_group_base_line/readme/USAGE.rst
Normal file
1
account_cash_basis_group_base_line/readme/USAGE.rst
Normal file
@@ -0,0 +1 @@
|
||||
Just Install the module. No extra configuration is needed.
|
||||
Reference in New Issue
Block a user