[FIX] account_financial_report: Open Items report now sorted by account code
This commit is contained in:
committed by
chaule97
parent
50270b6e00
commit
2dad961ebb
@@ -206,11 +206,18 @@ class OpenItemsReport(models.AbstractModel):
|
||||
|
||||
@api.model
|
||||
def _order_open_items_by_date(
|
||||
self, open_items_move_lines_data, show_partner_details, partners_data
|
||||
self,
|
||||
open_items_move_lines_data,
|
||||
show_partner_details,
|
||||
partners_data,
|
||||
accounts_data,
|
||||
):
|
||||
# We need to order by account code, partner_name and date
|
||||
accounts_data_sorted = sorted(accounts_data.items(), key=lambda x: x[1]["code"])
|
||||
account_ids_sorted = [account[0] for account in accounts_data_sorted]
|
||||
new_open_items = {}
|
||||
if not show_partner_details:
|
||||
for acc_id in open_items_move_lines_data.keys():
|
||||
for acc_id in account_ids_sorted:
|
||||
new_open_items[acc_id] = {}
|
||||
move_lines = []
|
||||
for prt_id in open_items_move_lines_data[acc_id]:
|
||||
@@ -219,7 +226,7 @@ class OpenItemsReport(models.AbstractModel):
|
||||
move_lines = sorted(move_lines, key=lambda k: (k["date"]))
|
||||
new_open_items[acc_id] = move_lines
|
||||
else:
|
||||
for acc_id in open_items_move_lines_data.keys():
|
||||
for acc_id in account_ids_sorted:
|
||||
new_open_items[acc_id] = {}
|
||||
for prt_id in sorted(
|
||||
open_items_move_lines_data[acc_id],
|
||||
@@ -262,7 +269,10 @@ class OpenItemsReport(models.AbstractModel):
|
||||
|
||||
total_amount = self._calculate_amounts(open_items_move_lines_data)
|
||||
open_items_move_lines_data = self._order_open_items_by_date(
|
||||
open_items_move_lines_data, show_partner_details, partners_data
|
||||
open_items_move_lines_data,
|
||||
show_partner_details,
|
||||
partners_data,
|
||||
accounts_data,
|
||||
)
|
||||
return {
|
||||
"doc_ids": [wizard_id],
|
||||
|
||||
Reference in New Issue
Block a user