tocmo0nlord/Odoo-18.0-20251222
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

[FIX] account_financial_report: trial balance hierarchy

Browse Source 
When the report is generated with 'hierarchy' option checked,
the group hierarchy values are not calculated correctly
This commit is contained in:
Ernesto Tejeda
2022-05-27 12:10:07 -04:00
committed by chaule97
parent dd455bacc4
commit 9efca74d7d
1 changed files with 31 additions and 58 deletions
Download Patch File Download Diff File Expand all files Collapse all files

89
account_financial_report/report/trial_balance.py
View File

@@ -498,59 +498,13 @@ class TrialBalanceReport(models.AbstractModel):
] += pl_initial_currency_balance ] += pl_initial_currency_balance
return total_amount, accounts_data, partners_data return total_amount, accounts_data, partners_data
def _get_hierarchy_groups( def _get_hierarchy_groups(self, group_ids, groups_data, foreign_currency):
self, group_ids, groups_data, old_groups_ids, foreign_currency
):
new_parents = False
for group_id in group_ids: for group_id in group_ids:
if groups_data[group_id]["parent_id"]: parent_id = groups_data[group_id]["parent_id"]
new_parents = True while parent_id:
nw_id = groups_data[group_id]["parent_id"] if parent_id not in groups_data.keys():
if nw_id in groups_data.keys(): group = self.env["account.group"].browse(parent_id)
groups_data[nw_id]["initial_balance"] += groups_data[group_id][ groups_data[group.id] = {
"initial_balance"
]
groups_data[nw_id]["debit"] += groups_data[group_id]["debit"]
groups_data[nw_id]["credit"] += groups_data[group_id]["credit"]
groups_data[nw_id]["balance"] += groups_data[group_id]["balance"]
groups_data[nw_id]["ending_balance"] += groups_data[group_id][
"ending_balance"
]
if foreign_currency:
groups_data[nw_id]["initial_currency_balance"] += groups_data[
group_id
]["initial_currency_balance"]
groups_data[nw_id]["ending_currency_balance"] += groups_data[
group_id
]["ending_currency_balance"]
else:
groups_data[nw_id] = {}
groups_data[nw_id]["initial_balance"] = groups_data[group_id][
"initial_balance"
]
groups_data[nw_id]["debit"] = groups_data[group_id]["debit"]
groups_data[nw_id]["credit"] = groups_data[group_id]["credit"]
groups_data[nw_id]["balance"] = groups_data[group_id]["balance"]
groups_data[nw_id]["ending_balance"] = groups_data[group_id][
"ending_balance"
]
if foreign_currency:
groups_data[nw_id]["initial_currency_balance"] = groups_data[
group_id
]["initial_currency_balance"]
groups_data[nw_id]["ending_currency_balance"] = groups_data[
group_id
]["ending_currency_balance"]
if new_parents:
nw_groups_ids = []
for group_id in list(groups_data.keys()):
if group_id not in old_groups_ids:
nw_groups_ids.append(group_id)
old_groups_ids.append(group_id)
groups = self.env["account.group"].browse(nw_groups_ids)
for group in groups:
groups_data[group.id].update(
{
"id": group.id, "id": group.id,
"code": group.code_prefix_start, "code": group.code_prefix_start,
"name": group.name, "name": group.name,
@@ -559,11 +513,29 @@ class TrialBalanceReport(models.AbstractModel):
"complete_code": group.complete_code, "complete_code": group.complete_code,
"account_ids": group.compute_account_ids.ids, "account_ids": group.compute_account_ids.ids,
"type": "group_type", "type": "group_type",
"initial_balance": 0,
"debit": 0,
"credit": 0,
"balance": 0,
"ending_balance": 0,
} }
) if foreign_currency:
groups_data = self._get_hierarchy_groups( groups_data[group.id].update(
nw_groups_ids, groups_data, old_groups_ids, foreign_currency initial_currency_balance=0,
) ending_currency_balance=0,
)
acc_keys = ["debit", "credit", "balance"]
acc_keys += ["initial_balance", "ending_balance"]
for acc_key in acc_keys:
groups_data[parent_id][acc_key] += groups_data[group_id][acc_key]
if foreign_currency:
groups_data[group_id]["initial_currency_balance"] += groups_data[
group_id
]["initial_currency_balance"]
groups_data[group_id]["ending_currency_balance"] += groups_data[
group_id
]["ending_currency_balance"]
parent_id = groups_data[parent_id]["parent_id"]
return groups_data return groups_data
def _get_groups_data(self, accounts_data, total_amount, foreign_currency): def _get_groups_data(self, accounts_data, total_amount, foreign_currency):
@@ -623,9 +595,10 @@ class TrialBalanceReport(models.AbstractModel):
account_id account_id
]["ending_currency_balance"] ]["ending_currency_balance"]
group_ids = list(groups_data.keys()) group_ids = list(groups_data.keys())
old_group_ids = list(groups_data.keys())
groups_data = self._get_hierarchy_groups( groups_data = self._get_hierarchy_groups(
group_ids, groups_data, old_group_ids, foreign_currency group_ids,
groups_data,
foreign_currency,
) )
return groups_data return groups_data