Skip to content
Snippets Groups Projects
Commit 9b350652 authored by Laurent Smet's avatar Laurent Smet
Browse files

[FIX] account: Partial revert of...

[FIX] account: Partial revert of https://github.com/odoo/odoo/commit/39fb6b2990e5d6fda0d5f12dbd64d26ef77b9d0c



It seems some ppl are sharing the same bank account accross multiple journals.

issues:
2527148
2527994
2527894
...

closes odoo/odoo#70676

Signed-off-by: default avataroco-odoo <oco-odoo@users.noreply.github.com>
parent 36866412
No related branches found
No related tags found
No related merge requests found
......@@ -656,6 +656,9 @@ class AccountJournal(models.Model):
def _get_journal_bank_account_balance(self, domain=None):
''' Get the bank balance of the current journal by filtering the journal items using the journal's accounts.
/!\ The current journal is not part of the applied domain. This is the expected behavior since we only want
a logic based on accounts.
:param domain: An additional domain to be applied on the account.move.line model.
:return: Tuple having balance expressed in journal's currency
along with the total number of move lines having the same account as of the journal's default account.
......@@ -670,7 +673,6 @@ class AccountJournal(models.Model):
('account_id', 'in', tuple(self.default_account_id.ids)),
('display_type', 'not in', ('line_section', 'line_note')),
('move_id.state', '!=', 'cancel'),
('journal_id', '=', self.id),
]
query = self.env['account.move.line']._where_calc(domain)
tables, where_clause, where_params = query.get_sql()
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment