odoo-addon-account-financial-report 16.0.1.15.2__py3-none-any.whl → 16.0.1.16.0__py3-none-any.whl

odoo/addons/account_financial_report/readme/ROADMAP.rst

@@ -5,3 +5,6 @@
 * It would be nice to have in reports a column indicating the
   state of the entries when the option "All Entries" is selected
   in "Target Moves" field in a wizard
+* Adapt reports so that the order of the columns displayed is dynamic
+* Remove duplicated options to show columns (it has not been made in a stable version so
+  as not to generate confusion in users)

odoo/addons/account_financial_report/report/abstract_report.py

@@ -1,4 +1,5 @@
 # Copyright 2020 ForgeFlow S.L. (https://www.forgeflow.com)
+# Copyright 2025 Tecnativa - Carlos Dauden
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
 from odoo import api, models
@@ -160,3 +161,17 @@ class AgedPartnerBalanceReport(models.AbstractModel):
             "debit",
             "amount_currency",
         ]
+
+    def _get_report_values(self, docids, data):
+        wizard = self.env[data["wizard_name"]].browse(data["wizard_id"])
+        res = {f"{c.expression_label}_visible": c.is_visible for c in wizard.column_ids}
+        res.update(
+            {
+                f"{c.expression_label}_limit": c.limit
+                for c in wizard.column_ids
+                if c.field_type == "string"
+            }
+        )
+        # Pass function to be called in report
+        res["limit_text"] = wizard._limit_text
+        return res

odoo/addons/account_financial_report/report/aged_partner_balance.py

@@ -403,6 +403,7 @@ class AgedPartnerBalanceReport(models.AbstractModel):
         return aged_partner_data
 
     def _get_report_values(self, docids, data):
+        res = super()._get_report_values(docids, data)
         wizard_id = data["wizard_id"]
         company = self.env["res.company"].browse(data["company_id"])
         company_id = data["company_id"]
@@ -440,18 +441,23 @@ class AgedPartnerBalanceReport(models.AbstractModel):
         aged_partner_data = self.with_context(
             age_partner_config=aged_partner_configuration
         )._calculate_percent(aged_partner_data)
-        return {
-            "doc_ids": [wizard_id],
-            "doc_model": "aged.partner.balance.report.wizard",
-            "docs": self.env["aged.partner.balance.report.wizard"].browse(wizard_id),
-            "company_name": company.display_name,
-            "currency_name": company.currency_id.name,
-            "date_at": date_at,
-            "only_posted_moves": only_posted_moves,
-            "aged_partner_balance": aged_partner_data,
-            "show_move_lines_details": show_move_line_details,
-            "age_partner_config": aged_partner_configuration,
-        }
+        res.update(
+            {
+                "doc_ids": [wizard_id],
+                "doc_model": "aged.partner.balance.report.wizard",
+                "docs": self.env["aged.partner.balance.report.wizard"].browse(
+                    wizard_id
+                ),
+                "company_name": company.display_name,
+                "currency_name": company.currency_id.name,
+                "date_at": date_at,
+                "only_posted_moves": only_posted_moves,
+                "aged_partner_balance": aged_partner_data,
+                "show_move_lines_details": show_move_line_details,
+                "age_partner_config": aged_partner_configuration,
+            }
+        )
+        return res
 
     def _get_ml_fields(self):
         return self.COMMON_ML_FIELDS + [

odoo/addons/account_financial_report/report/general_ledger.py

@@ -376,7 +376,6 @@ class GeneralLedgerReport(models.AbstractModel):
             domain += [("move_id.state", "=", "posted")]
         else:
             domain += [("move_id.state", "in", ["posted", "draft"])]
-
         if cost_center_ids:
             domain += [("analytic_account_ids", "in", cost_center_ids)]
         return domain
@@ -749,6 +748,7 @@ class GeneralLedgerReport(models.AbstractModel):
 
     # flake8: noqa: C901
     def _get_report_values(self, docids, data):
+        res = super()._get_report_values(docids, data)
         wizard_id = data["wizard_id"]
         company = self.env["res.company"].browse(data["company_id"])
         company_id = data["company_id"]
@@ -877,29 +877,32 @@ class GeneralLedgerReport(models.AbstractModel):
             if not gl_item["currency_id"] and len(fin_bal_currency_ids) == 1:
                 fin_bal_currency_id = fin_bal_currency_ids[0]
             gl_item["fin_bal_currency_id"] = fin_bal_currency_id
-        return {
-            "doc_ids": [wizard_id],
-            "doc_model": "general.ledger.report.wizard",
-            "docs": self.env["general.ledger.report.wizard"].browse(wizard_id),
-            "foreign_currency": data["foreign_currency"],
-            "company_name": company.display_name,
-            "company_currency": company.currency_id,
-            "currency_name": company.currency_id.name,
-            "date_from": data["date_from"],
-            "date_to": data["date_to"],
-            "only_posted_moves": data["only_posted_moves"],
-            "hide_account_at_0": data["hide_account_at_0"],
-            "show_cost_center": data["show_cost_center"],
-            "general_ledger": general_ledger,
-            "accounts_data": accounts_data,
-            "journals_data": journals_data,
-            "full_reconcile_data": full_reconcile_data,
-            "taxes_data": taxes_data,
-            "centralize": centralize,
-            "analytic_data": analytic_data,
-            "filter_partner_ids": True if partner_ids else False,
-            "currency_model": self.env["res.currency"],
-        }
+        res.update(
+            {
+                "doc_ids": [wizard_id],
+                "doc_model": "general.ledger.report.wizard",
+                "docs": self.env["general.ledger.report.wizard"].browse(wizard_id),
+                "foreign_currency": data["foreign_currency"],
+                "company_name": company.display_name,
+                "company_currency": company.currency_id,
+                "currency_name": company.currency_id.name,
+                "date_from": data["date_from"],
+                "date_to": data["date_to"],
+                "only_posted_moves": data["only_posted_moves"],
+                "hide_account_at_0": data["hide_account_at_0"],
+                "show_cost_center": data["show_cost_center"],
+                "general_ledger": general_ledger,
+                "accounts_data": accounts_data,
+                "journals_data": journals_data,
+                "full_reconcile_data": full_reconcile_data,
+                "taxes_data": taxes_data,
+                "centralize": centralize,
+                "analytic_data": analytic_data,
+                "filter_partner_ids": True if partner_ids else False,
+                "currency_model": self.env["res.currency"],
+            }
+        )
+        return res
 
     def _get_ml_fields(self):
         return self.COMMON_ML_FIELDS + [

odoo/addons/account_financial_report/report/journal_ledger.py

@@ -9,6 +9,7 @@ from odoo import models
 
 
 class JournalLedgerReport(models.AbstractModel):
+    _inherit = "report.account_financial_report.abstract_report"
     _name = "report.account_financial_report.journal_ledger"
     _description = "Journal Ledger Report"
 
@@ -298,6 +299,7 @@ class JournalLedgerReport(models.AbstractModel):
         return journals_taxes_data_2
 
     def _get_report_values(self, docids, data):
+        res = super()._get_report_values(docids, data)
         wizard_id = data["wizard_id"]
         wizard = self.env["journal.ledger.report.wizard"].browse(wizard_id)
         company = self.env["res.company"].browse(data["company_id"])
@@ -351,25 +353,28 @@ class JournalLedgerReport(models.AbstractModel):
             if journal_id in journal_totals.keys():
                 for item in ["debit", "credit"]:
                     journal_ledger_data[item] += journal_totals[journal_id][item]
-        return {
-            "doc_ids": [wizard_id],
-            "doc_model": "journal.ledger.report.wizard",
-            "docs": self.env["journal.ledger.report.wizard"].browse(wizard_id),
-            "group_option": data["group_option"],
-            "foreign_currency": data["foreign_currency"],
-            "with_account_name": data["with_account_name"],
-            "company_name": company.display_name,
-            "currency_name": company.currency_id.name,
-            "date_from": data["date_from"],
-            "date_to": data["date_to"],
-            "move_target": data["move_target"],
-            "with_auto_sequence": data["with_auto_sequence"],
-            "account_ids_data": account_ids_data,
-            "partner_ids_data": partner_ids_data,
-            "currency_ids_data": currency_ids_data,
-            "move_ids_data": move_ids_data,
-            "tax_line_data": tax_line_ids_data,
-            "move_line_ids_taxes_data": move_line_ids_taxes_data,
-            "Journal_Ledgers": journal_ledgers_data,
-            "Moves": moves_data,
-        }
+        res.update(
+            {
+                "doc_ids": [wizard_id],
+                "doc_model": "journal.ledger.report.wizard",
+                "docs": self.env["journal.ledger.report.wizard"].browse(wizard_id),
+                "group_option": data["group_option"],
+                "foreign_currency": data["foreign_currency"],
+                "with_account_name": data["with_account_name"],
+                "company_name": company.display_name,
+                "currency_name": company.currency_id.name,
+                "date_from": data["date_from"],
+                "date_to": data["date_to"],
+                "move_target": data["move_target"],
+                "with_auto_sequence": data["with_auto_sequence"],
+                "account_ids_data": account_ids_data,
+                "partner_ids_data": partner_ids_data,
+                "currency_ids_data": currency_ids_data,
+                "move_ids_data": move_ids_data,
+                "tax_line_data": tax_line_ids_data,
+                "move_line_ids_taxes_data": move_line_ids_taxes_data,
+                "Journal_Ledgers": journal_ledgers_data,
+                "Moves": moves_data,
+            }
+        )
+        return res

odoo/addons/account_financial_report/report/open_items.py

@@ -159,7 +159,7 @@ class OpenItemsReport(models.AbstractModel):
                     "partner_name": partner.name or "",
                     "ref_label": ref_label,
                     "journal_id": move_line["journal_id"][0],
-                    "move_name": move_line["move_id"][1],
+                    "move_name": move_line["move_name"],
                     "entry_id": move_line["move_id"][0],
                     "currency_id": move_line["currency_id"][0]
                     if move_line["currency_id"]
@@ -243,6 +243,7 @@ class OpenItemsReport(models.AbstractModel):
         return new_open_items
 
     def _get_report_values(self, docids, data):
+        res = super()._get_report_values(docids, data)
         wizard_id = data["wizard_id"]
         company = self.env["res.company"].browse(data["company_id"])
         company_id = data["company_id"]
@@ -277,24 +278,27 @@ class OpenItemsReport(models.AbstractModel):
             partners_data,
             accounts_data,
         )
-        return {
-            "doc_ids": [wizard_id],
-            "doc_model": "open.items.report.wizard",
-            "docs": self.env["open.items.report.wizard"].browse(wizard_id),
-            "foreign_currency": data["foreign_currency"],
-            "show_partner_details": data["show_partner_details"],
-            "company_name": company.display_name,
-            "currency_name": company.currency_id.name,
-            "date_at": date_at_object.strftime("%d/%m/%Y"),
-            "hide_account_at_0": data["hide_account_at_0"],
-            "target_move": data["target_move"],
-            "journals_data": journals_data,
-            "partners_data": partners_data,
-            "accounts_data": accounts_data,
-            "total_amount": total_amount,
-            "Open_Items": open_items_move_lines_data,
-            "grouped_by": grouped_by,
-        }
+        res.update(
+            {
+                "doc_ids": [wizard_id],
+                "doc_model": "open.items.report.wizard",
+                "docs": self.env["open.items.report.wizard"].browse(wizard_id),
+                "foreign_currency": data["foreign_currency"],
+                "show_partner_details": data["show_partner_details"],
+                "company_name": company.display_name,
+                "currency_name": company.currency_id.name,
+                "date_at": date_at_object.strftime("%d/%m/%Y"),
+                "hide_account_at_0": data["hide_account_at_0"],
+                "target_move": data["target_move"],
+                "journals_data": journals_data,
+                "partners_data": partners_data,
+                "accounts_data": accounts_data,
+                "total_amount": total_amount,
+                "Open_Items": open_items_move_lines_data,
+                "grouped_by": grouped_by,
+            }
+        )
+        return res
 
     def _get_ml_fields(self):
         return self.COMMON_ML_FIELDS + [
@@ -306,4 +310,5 @@ class OpenItemsReport(models.AbstractModel):
             "amount_residual_currency",
             "debit",
             "amount_currency",
+            "move_name",
         ]