odoo-addon-account-financial-report 15.0.1.1.0.1__py3-none-any.whl → 18.0.1.4.4.3__py3-none-any.whl

odoo/addons/account_financial_report/menuitems.xml

@@ -4,7 +4,7 @@
         parent="account.menu_finance_reports"
         id="menu_oca_reports"
         name="OCA accounting reports"
-        groups="account.group_account_manager,account.group_account_user"
+        groups="account.group_account_user"
     />
     <menuitem
         parent="menu_oca_reports"

odoo/addons/account_financial_report/models/__init__.py

@@ -1,4 +1,6 @@
+from . import account_age_report_configuration
 from . import account_group
 from . import account
 from . import account_move_line
 from . import ir_actions_report
+from . import res_config_settings

odoo/addons/account_financial_report/models/account_age_report_configuration.py

@@ -0,0 +1,49 @@
+# Copyright 2023 Ernesto García
+# Copyright 2023 Carolina Fernandez
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+from odoo import api, fields, models
+from odoo.exceptions import ValidationError
+
+
+class AccountAgeReportConfiguration(models.Model):
+    _name = "account.age.report.configuration"
+    _description = "Model to set intervals for Age partner balance report"
+
+    name = fields.Char(required=True)
+    company_id = fields.Many2one(
+        "res.company", default=lambda self: self.env.company, readonly=True
+    )
+    line_ids = fields.One2many(
+        "account.age.report.configuration.line", "account_age_report_config_id"
+    )
+
+    @api.constrains("line_ids")
+    def _check_line_ids(self):
+        for rec in self:
+            if not rec.line_ids:
+                raise ValidationError(self.env._("Must complete Configuration Lines"))
+
+
+class AccountAgeReportConfigurationLine(models.Model):
+    _name = "account.age.report.configuration.line"
+    _description = "Model to set interval lines for Age partner balance report"
+
+    name = fields.Char(required=True)
+    account_age_report_config_id = fields.Many2one("account.age.report.configuration")
+    inferior_limit = fields.Integer()
+
+    @api.constrains("inferior_limit")
+    def _check_inferior_limit(self):
+        for rec in self:
+            if rec.inferior_limit <= 0:
+                raise ValidationError(
+                    self.env._("Inferior Limit must be greather than zero")
+                )
+
+    _sql_constraints = [
+        (
+            "unique_name_config_combination",
+            "UNIQUE(name,account_age_report_config_id)",
+            "Name must be unique per report configuration",
+        )
+    ]

odoo/addons/account_financial_report/models/account_group.py

@@ -2,6 +2,7 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
 from odoo import api, fields, models
+from odoo.tools import SQL
 
 
 class AccountGroup(models.Model):
@@ -12,13 +13,17 @@ class AccountGroup(models.Model):
     )
     level = fields.Integer(compute="_compute_level", recursive=True)
     account_ids = fields.One2many(
-        comodel_name="account.account", inverse_name="group_id", string="Accounts"
+        comodel_name="account.account",
+        compute="_compute_account_ids",
+        string="Accounts",
+        store=False,
     )
     compute_account_ids = fields.Many2many(
         "account.account",
+        recursive=True,
         compute="_compute_group_accounts",
         string="Compute accounts",
-        store=True,
+        store=False,
     )
     complete_name = fields.Char(
         "Full Name", compute="_compute_complete_name", recursive=True
@@ -30,20 +35,60 @@ class AccountGroup(models.Model):
     @api.depends("name", "parent_id.complete_name")
     def _compute_complete_name(self):
         """Forms complete name of location from parent location to child location."""
-        if self.parent_id.complete_name:
-            self.complete_name = "{}/{}".format(self.parent_id.complete_name, self.name)
-        else:
-            self.complete_name = self.name
+        for group in self:
+            if group.parent_id.complete_name:
+                group.complete_name = f"{group.parent_id.complete_name}/{group.name}"
+            else:
+                group.complete_name = group.name
+
+    @api.depends("code_prefix_start", "code_prefix_end")
+    def _compute_account_ids(self):
+        """Retrieves every account from `self`.
+        In Odoo 18 the group_id on account is not stored so it raises
+        an error the one2many account_ids with inverse name group_id."""
+        group_ids = self.ids
+        self.account_ids = self.env["account.account"]
+        if not group_ids:
+            return
+        group_ids = SQL(",".join(map(str, group_ids)))
+        results = self.env.execute_query(
+            SQL(
+                """
+        SELECT
+        agroup.id AS group_id,
+        STRING_AGG(DISTINCT account.id::text, ', ') as account_ids
+        FROM  account_group agroup
+        inner join account_account account
+        ON agroup.code_prefix_start <= LEFT(%(code_store)s->>agroup.company_id::text,
+        char_length(agroup.code_prefix_start))
+        AND agroup.code_prefix_end >= LEFT(%(code_store)s->>agroup.company_id::text,
+        char_length(agroup.code_prefix_end))
+        WHERE agroup.id IN (%(group_ids)s)
+        GROUP BY agroup.id
+            """,
+                code_store=SQL.identifier("account", "code_store"),
+                group_ids=group_ids,
+            )
+        )
+        group_by_code = dict(results)
+        if not group_by_code:
+            return
+        for record in self:
+            if group_by_code.get(record.id, ""):
+                record.account_ids = list(
+                    map(int, group_by_code.get(record.id, "").split(", "))
+                )
 
     @api.depends("code_prefix_start", "parent_id.complete_code")
     def _compute_complete_code(self):
         """Forms complete code of location from parent location to child location."""
-        if self.parent_id.complete_code:
-            self.complete_code = "{}/{}".format(
-                self.parent_id.complete_code, self.code_prefix_start
-            )
-        else:
-            self.complete_code = self.code_prefix_start
+        for group in self:
+            if group.parent_id.complete_code:
+                group.complete_code = (
+                    f"{group.parent_id.complete_code}/{group.code_prefix_start}"
+                )
+            else:
+                group.complete_code = group.code_prefix_start
 
     @api.depends("parent_id", "parent_id.level")
     def _compute_level(self):
@@ -54,16 +99,10 @@ class AccountGroup(models.Model):
                 group.level = group.parent_id.level + 1
 
     @api.depends(
-        "code_prefix_start",
-        "account_ids",
-        "account_ids.code",
-        "group_child_ids",
-        "group_child_ids.account_ids.code",
+        "group_child_ids.compute_account_ids",
     )
     def _compute_group_accounts(self):
-        account_obj = self.env["account.account"]
-        accounts = account_obj.search([])
-        for group in self:
-            prefix = group.code_prefix_start if group.code_prefix_start else group.name
-            gr_acc = accounts.filtered(lambda a: a.code.startswith(prefix)).ids
-            group.compute_account_ids = [(6, 0, gr_acc)]
+        for one in self:
+            one.compute_account_ids = (
+                one.account_ids | one.group_child_ids.compute_account_ids
+            )

odoo/addons/account_financial_report/models/account_move_line.py

@@ -1,11 +1,41 @@
 # Copyright 2019 ACSONE SA/NV (<http://acsone.eu>)
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).-
-from odoo import api, models
+from collections import defaultdict
+
+from odoo import api, fields, models
+from odoo.fields import Command
 
 
 class AccountMoveLine(models.Model):
     _inherit = "account.move.line"
 
+    analytic_account_ids = fields.Many2many(
+        "account.analytic.account", compute="_compute_analytic_account_ids", store=True
+    )
+
+    @api.depends("analytic_distribution")
+    def _compute_analytic_account_ids(self):
+        # Prefetch all involved analytic accounts
+        batch_by_analytic_account = defaultdict(lambda: self.env["account.move.line"])
+        for record in self.filtered("analytic_distribution"):
+            # NB: ``analytic_distribution`` is a JSON field where keys can be either
+            # 'account.id' or 'account.id,account.id'
+            # Eg: https://github.com/odoo/odoo/blob/8479b4e/addons/sale/models/account_move_line.py#L158
+            for key in record.analytic_distribution:
+                for account_id in map(int, key.split(",")):
+                    batch_by_analytic_account[account_id] += record
+        existing_account_ids = set(
+            self.env["account.analytic.account"]
+            .browse(batch_by_analytic_account)
+            .exists()
+            .ids
+        )
+        # Store them
+        self.analytic_account_ids = [Command.clear()]
+        for account_id, records in batch_by_analytic_account.items():
+            if account_id in existing_account_ids:
+                records.analytic_account_ids = [Command.link(account_id)]
+
     def init(self):
         """
             The join between accounts_partners subquery and account_move_line
@@ -32,9 +62,9 @@ class AccountMoveLine(models.Model):
             )
 
     @api.model
-    def search_count(self, args):
+    def search_count(self, domain, limit=None):
         # In Big DataBase every time you change the domain widget this method
         # takes a lot of time. This improves performance
         if self.env.context.get("skip_search_count"):
             return 0
-        return super(AccountMoveLine, self).search_count(args)
+        return super().search_count(domain, limit=limit)

odoo/addons/account_financial_report/models/ir_actions_report.py

@@ -13,13 +13,15 @@ class IrActionsReport(models.Model):
         return dict(self.env.context or {}, lang=lang) if lang else False
 
     @api.model
-    def _render_qweb_html(self, docids, data=None):
+    def _render_qweb_html(self, report_ref, docids, data=None):
         context = self._prepare_account_financial_report_context(data)
         obj = self.with_context(**context) if context else self
-        return super(IrActionsReport, obj)._render_qweb_html(docids, data)
+        return super(IrActionsReport, obj)._render_qweb_html(
+            report_ref, docids, data=data
+        )
 
     @api.model
-    def _render_xlsx(self, docids, data):
+    def _render_xlsx(self, report_ref, docids, data=None):
         context = self._prepare_account_financial_report_context(data)
         obj = self.with_context(**context) if context else self
-        return super(IrActionsReport, obj)._render_xlsx(docids, data)
+        return super(IrActionsReport, obj)._render_xlsx(report_ref, docids, data=data)

odoo/addons/account_financial_report/models/res_config_settings.py

@@ -0,0 +1,36 @@
+# Copyright 2023 Tecnativa - Carolina Fernandez
+# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl.html).
+
+from odoo import api, fields, models
+
+
+class ResConfigSettings(models.TransientModel):
+    _inherit = "res.config.settings"
+
+    age_partner_config_id = fields.Many2one(
+        "account.age.report.configuration",
+        string="Intervals configuration",
+    )
+
+    def set_values(self):
+        self.env["ir.default"].sudo().set(
+            "aged.partner.balance.report.wizard",
+            "age_partner_config_id",
+            self.age_partner_config_id.id,
+            company_id=self.env.company.id,
+        )
+        return super().set_values()
+
+    @api.model
+    def get_values(self):
+        res = super().get_values()
+        res.update(
+            age_partner_config_id=self.env["ir.default"]
+            .sudo()
+            ._get(
+                "aged.partner.balance.report.wizard",
+                "age_partner_config_id",
+                company_id=self.env.company.id,
+            )
+        )
+        return res

odoo/addons/account_financial_report/readme/CONFIGURE.md

@@ -0,0 +1,26 @@
+To configure dynamic intervals for Aged Partner Balance you need to:
+
+Go on 'Settings' -> 'Invoicing' -> 'OCA Aged Report Configuration'.
+
+Click on option 'Configurations' and create new record.
+
+Create new interval.
+The name established on line will be the column to display in Aged Partner Balance.
+Inferior limit established on line is the interval
+
+Example of configuration inferior limit:
+
+-> 15
+-> 30
+-> 60
+
+It means the first interval is from 0 to 15, the second from 16 to 30, and the third is 61+.
+
+Go on 'Invoicing' -> 'Reporting' -> 'OCA accounting reports' -> 'Aged Partner Balance'
+
+When wizard is open, you need to select your interval configuration and print report.
+
+If you want to get default interval configuration any time you wish to print Aged Partner Report,
+you can set default interval configuration per company in:
+
+'Settings' -> 'Invoicing' -> 'OCA Aged Report Configuration'.

odoo/addons/account_financial_report/readme/CONTRIBUTORS.md

@@ -0,0 +1,38 @@
+- Jordi Ballester \<<jordi.ballester@forgeflow.com>\>
+- Yannick Vaucher \<<yannick.vaucher@camptocamp.com>\>
+- Simone Orsi \<<simone.orsi@abstract.com>\>
+- Leonardo Pistone \<<leonardo.pistone@camptocamp.com>\>
+- Damien Crier \<<damien.crier@camptocamp.com>\>
+- Andrea Stirpe \<<a.stirpe@onestein.nl>\>
+- Thomas Rehn \<<thomas.rehn@initos.com>\>
+- Andrea Gallina \<<4everamd@gmail.com>\>
+- Robert Rottermann \<<robert@redcor.ch>\>
+- Ciro Urselli \<<c.urselli@apuliasoftware.it>\>
+- Francesco Apruzzese \<<opencode@e-ware.org>\>
+- Lorenzo Battistini \<<https://github.com/eLBati>\>
+- Julien Coux \<<julien.coux@camptocamp.com>\>
+- Akim Juillerat \<<akim.juillerat@camptocamp.com>\>
+- Alexis de Lattre \<<alexis@via.ecp.fr>\>
+- Mihai Fekete \<<feketemihai@gmail.com>\>
+- Miquel Ra??ch \<<miquel.raich@forgeflow.com>\>
+- Joan Sisquella \<<joan.sisquella@forgeflow.com>\>
+- [Tecnativa](https://www.tecnativa.com):
+  - Pedro M. Baeza
+  - Sergio Teruel
+  - Ernesto Tejeda
+  - João Marques
+  - Alexandre D. D??az
+  - V??ctor Mart??nez
+  - Carolina Fernandez
+- [Sygel](https://www.sygel.es):
+  - Harald Panten
+  - Valentin Vinagre
+- Lois Rilo \<<lois.rilo@forgeflow.com>\>
+- Saran Lim. \<<saranl@ecosoft.co.th>\>
+- Omar Casti??eira \<<omar@comunitea.com>\>
+- Chau Le \<<chaulb@trobz.com>\>
+- [Stesi Consulting](https://www.stesi.consulting):
+  - Michele Di Croce \<<dicroce.m@stesi.consulting>\>
+  
+Much of the work in this module was done at a sprint in Sorrento, Italy
+in April 2016.

odoo/addons/account_financial_report/readme/CREDITS.md

@@ -0,0 +1 @@
+The migration of this module from 17.0 to 18.0 was financially supported by Camptocamp.

odoo/addons/account_financial_report/readme/DESCRIPTION.md

@@ -0,0 +1,21 @@
+This module adds a set of financial reports. They are accessible under
+Invoicing / Reporting / OCA accounting reports.
+
+- General ledger
+- Trial Balance
+- Open Items
+- Aged Partner Balance
+- VAT Report
+- Journal Ledger
+
+Currently General ledger, Trial Balance and Open Items are fully
+compatible with a foreign currency set up in account in order to display
+balances. Moreover, any foreign currency used in account move lines is
+properly shown.
+
+In case that in an account has not been configured a second currency
+foreign currency balances are not available.
+
+Invoicing / Settings / Invoicing / OCA Aged Report Configuration you will be able to set
+dynamic intervals that will appear on the Aged Partner Balance.
+For further information, check CONFIGURE.rst

odoo/addons/account_financial_report/readme/HISTORY.md

@@ -0,0 +1,16 @@
+## 11.0.2.5.0 (2019-04-26)
+
+- In the Trial Balance you have an option to hide parent hierarchy
+  levels
+
+## 11.0.2.4.1 (2019-01-08)
+
+- Handle better multicompany behaviour
+- Improve how title appears in the reports
+- Improve performance in General Ledger
+
+## 11.0.2.3.1 (2018-11-29)
+
+- In the Trial Balance you can apply a filter by hierarchy levels
+- In the General Ledger you can apply a filter by Analytic Tag
+- In the Journal Ledger the field 'Journal' is now optional

odoo/addons/account_financial_report/readme/ROADMAP.md

@@ -0,0 +1,6 @@
+- 'VAT Report' is valid only for cases where it's met that for each Tax
+  defined: all the "Account tags" of all the 'Repartition for Invoices'
+  or 'Repartition for Credit Notes' are different.
+- 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

odoo/addons/account_financial_report/report/abstract_report.py

@@ -7,6 +7,16 @@ from odoo import api, models
 class AgedPartnerBalanceReport(models.AbstractModel):
     _name = "report.account_financial_report.abstract_report"
     _description = "Abstract Report"
+    COMMON_ML_FIELDS = [
+        "account_id",
+        "partner_id",
+        "journal_id",
+        "date",
+        "ref",
+        "id",
+        "move_id",
+        "name",
+    ]
 
     @api.model
     def _get_move_lines_domain_not_reconciled(
@@ -56,6 +66,8 @@ class AgedPartnerBalanceReport(models.AbstractModel):
         company_id,
         partner_ids,
         only_posted_moves,
+        debit_amount_currency,
+        credit_amount_currency,
     ):
         debit_ids = set(debit_ids)
         credit_ids = set(credit_ids)
@@ -68,24 +80,8 @@ class AgedPartnerBalanceReport(models.AbstractModel):
         new_domain = self._get_new_move_lines_domain(
             new_ml_ids, account_ids, company_id, partner_ids, only_posted_moves
         )
-        ml_fields = [
-            "id",
-            "name",
-            "date",
-            "move_id",
-            "journal_id",
-            "account_id",
-            "partner_id",
-            "amount_residual",
-            "date_maturity",
-            "ref",
-            "debit",
-            "credit",
-            "reconciled",
-            "currency_id",
-            "amount_currency",
-            "amount_residual_currency",
-        ]
+        company_currency = self.env["res.company"].browse(company_id).currency_id
+        ml_fields = self._get_ml_fields()
         new_move_lines = self.env["account.move.line"].search_read(
             domain=new_domain, fields=ml_fields
         )
@@ -93,13 +89,45 @@ class AgedPartnerBalanceReport(models.AbstractModel):
         for move_line in move_lines:
             ml_id = move_line["id"]
             if ml_id in debit_ids:
-                move_line["amount_residual"] += debit_amount[ml_id]
+                if move_line.get("amount_residual", False):
+                    move_line["amount_residual"] += debit_amount[ml_id]
+                else:
+                    move_line["amount_residual"] = debit_amount[ml_id]
+                if move_line.get("amount_residual_currency", False):
+                    move_line["amount_residual_currency"] += debit_amount_currency[
+                        ml_id
+                    ]
+                else:
+                    move_line["amount_residual_currency"] = debit_amount_currency[ml_id]
             if ml_id in credit_ids:
-                move_line["amount_residual"] -= credit_amount[ml_id]
+                if move_line.get("amount_residual", False):
+                    move_line["amount_residual"] -= credit_amount[ml_id]
+                else:
+                    move_line["amount_residual"] = -credit_amount[ml_id]
+                if move_line.get("amount_residual_currency", False):
+                    move_line["amount_residual_currency"] -= credit_amount_currency[
+                        ml_id
+                    ]
+                else:
+                    move_line["amount_residual_currency"] = -credit_amount_currency[
+                        ml_id
+                    ]
+            # Set amount_currency=0 to keep the same behaviour as in v13
+            # Conditions: if there is no curency_id defined or it is equal
+            # to the company's curency_id
+            if "amount_currency" in move_line and (
+                "currency_id" not in move_line
+                or move_line["currency_id"] == company_currency.id
+            ):
+                move_line["amount_currency"] = 0
         return move_lines
 
     def _get_accounts_data(self, accounts_ids):
-        accounts = self.env["account.account"].browse(accounts_ids)
+        accounts = (
+            self.env["account.account"]
+            .with_context(active_test=False)
+            .browse(accounts_ids)
+        )
         accounts_data = {}
         for account in accounts:
             accounts_data.update(
@@ -110,7 +138,7 @@ class AgedPartnerBalanceReport(models.AbstractModel):
                         "name": account.name,
                         "hide_account": False,
                         "group_id": account.group_id.id,
-                        "currency_id": account.currency_id or False,
+                        "currency_id": account.currency_id.id,
                         "currency_name": account.currency_id.name,
                         "centralized": account.centralized,
                     }
@@ -119,8 +147,30 @@ class AgedPartnerBalanceReport(models.AbstractModel):
         return accounts_data
 
     def _get_journals_data(self, journals_ids):
-        journals = self.env["account.journal"].browse(journals_ids)
+        journals = (
+            self.env["account.journal"]
+            .with_context(active_test=False)
+            .search_fetch([("id", "in", journals_ids)], ["code"])
+        )
         journals_data = {}
         for journal in journals:
             journals_data.update({journal.id: {"id": journal.id, "code": journal.code}})
         return journals_data
+
+    def _get_ml_fields(self):
+        return self.COMMON_ML_FIELDS + [
+            "amount_residual",
+            "reconciled",
+            "currency_id",
+            "credit",
+            "date_maturity",
+            "amount_residual_currency",
+            "debit",
+            "amount_currency",
+        ]
+
+    def _get_report_values(self, docids, data):
+        wizard = self.env[data["wizard_name"]].browse(data["wizard_id"])
+        return {
+            "limit_text": wizard._limit_text,
+        }