From 9b3506523659b06440b46664eeb83952cc30694e Mon Sep 17 00:00:00 2001
From: Laurent Smet <las@odoo.com>
Date: Tue, 11 May 2021 12:59:42 +0000
Subject: [PATCH] [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: oco-odoo <oco-odoo@users.noreply.github.com>
---
addons/account/models/account_journal.py | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/addons/account/models/account_journal.py b/addons/account/models/account_journal.py
index 5819a53e9706..ec2ea3504edb 100644
--- a/addons/account/models/account_journal.py
+++ b/addons/account/models/account_journal.py
@@ -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()
--
GitLab