From 3e8ec786a1e27e4b56ea8fcfb45596cca372c297 Mon Sep 17 00:00:00 2001
From: Simone Rubino <sir@takobi.online>
Date: Thu, 2 Mar 2023 10:23:22 +0100
Subject: [PATCH] [IMP] l10n_it_account_balance_report: Allow to hide accounts
 codes

---
 .../report/account_balance_report.py          |   1 +
 .../templates/account_balance_report.xml      |   2 +-
 .../tests/__init__.py                         |   3 +
 .../tests/test_balance_report.py              | 114 ++++++++++++++++++
 .../wizard/wizard_account_balance_report.py   |   2 +
 .../wizard/wizard_account_balance_report.xml  |   1 +
 6 files changed, 122 insertions(+), 1 deletion(-)
 create mode 100644 l10n_it_account_balance_report/tests/__init__.py
 create mode 100644 l10n_it_account_balance_report/tests/test_balance_report.py

diff --git a/l10n_it_account_balance_report/report/account_balance_report.py b/l10n_it_account_balance_report/report/account_balance_report.py
index 0addce3f2e5c..3f4329cf7a5c 100644
--- a/l10n_it_account_balance_report/report/account_balance_report.py
+++ b/l10n_it_account_balance_report/report/account_balance_report.py
@@ -21,6 +21,7 @@ class ReportAccountBalanceReport(models.TransientModel):
         [('profit_loss', "Profit & Loss"),
          ('balance_sheet', "Balance Sheet")],
     )
+    hide_accounts_codes = fields.Boolean()
     left_col_name = fields.Char()
     right_col_name = fields.Char()
     section_credit_ids = fields.One2many(
diff --git a/l10n_it_account_balance_report/report/templates/account_balance_report.xml b/l10n_it_account_balance_report/report/templates/account_balance_report.xml
index 40cda0fe5a37..2007f0f17e1e 100644
--- a/l10n_it_account_balance_report/report/templates/account_balance_report.xml
+++ b/l10n_it_account_balance_report/report/templates/account_balance_report.xml
@@ -160,7 +160,7 @@
         <div class="act_as_row lines">
             <!-- Account code -->
             <div class="act_as_cell left">
-                <t t-if="line.account_id">
+                <t t-if="line.account_id and not o.hide_accounts_codes">
                     <t t-set="res_model" t-value="'account.account'"/>
                     <span>
                         <a t-att-data-active-id="line.account_id.id"
diff --git a/l10n_it_account_balance_report/tests/__init__.py b/l10n_it_account_balance_report/tests/__init__.py
new file mode 100644
index 000000000000..8e0fe90f8119
--- /dev/null
+++ b/l10n_it_account_balance_report/tests/__init__.py
@@ -0,0 +1,3 @@
+#  License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from . import test_balance_report
diff --git a/l10n_it_account_balance_report/tests/test_balance_report.py b/l10n_it_account_balance_report/tests/test_balance_report.py
new file mode 100644
index 000000000000..6a9ba63dc648
--- /dev/null
+++ b/l10n_it_account_balance_report/tests/test_balance_report.py
@@ -0,0 +1,114 @@
+#  Copyright 2023 Simone Rubino - TAKOBI
+#  License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from datetime import timedelta
+
+from odoo import tests
+from odoo.addons.account.tests.account_test_classes import AccountingTestCase
+from odoo.tests import Form
+
+
+class TestBalanceReport (tests.SavepointCase, AccountingTestCase):
+
+    @classmethod
+    def _create_invoice(cls, partner, products=None, post=False):
+        """Get an invoice for `partner`, containing `products`.
+
+        If `post`, the invoice is opened.
+        """
+        if products is None:
+            products = cls.env['product.product'].browse()
+
+        invoice_form = Form(cls.env['account.invoice'])
+        invoice_form.partner_id = partner
+        for product in products:
+            with invoice_form.invoice_line_ids.new() as line:
+                line.product_id = product
+        invoice = invoice_form.save()
+
+        if post:
+            invoice.action_invoice_open()
+        return invoice
+
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.customer = cls.env['res.partner'].create({
+            'name': "Test Customer",
+            'customer': True,
+        })
+
+        cls.product = cls.env['product.product'].create({
+            'name': "Test Product",
+        })
+
+        cls.posted_invoice = cls._create_invoice(
+            cls.customer,
+            products=cls.product,
+            post=True,
+        )
+
+    def _get_report_content(self, wizard_values):
+        """Get the PDF content from the wizard created with `wizard_values`."""
+        # Get the Report Action from the Wizard
+        wiz = self.env['trial.balance.report.wizard'].create(wizard_values)
+        report_action = wiz.button_export_pdf()
+
+        # Get the Report from the Report Action
+        report_name = report_action['report_name']
+        context = report_action['context']
+        report_ids = context['active_ids']
+        report = self.env['ir.actions.report']._get_report_from_name(report_name)
+
+        # Render the Report
+        report_content, report_type = report \
+            .with_context(context) \
+            .render_qweb_pdf(report_ids)
+        report_content = report_content.decode()
+        return report_content
+
+    def test_hide_accounts_codes(self):
+        """The Account Code is hidden
+        when `hide_accounts_codes` is enabled.
+        """
+        # Arrange
+        invoice = self.posted_invoice
+        account = invoice.invoice_line_ids.account_id
+        # pre-condition: An Invoice is posted
+        self.assertEqual(invoice.state, 'open')
+
+        # Act: Print the Report containing the Invoice,
+        # enabling `hide_accounts_codes`
+        one_day = timedelta(days=1)
+        report_content = self._get_report_content({
+            'account_balance_report_type': 'profit_loss',
+            'hide_accounts_codes': True,
+            'date_from': invoice.date_invoice - one_day,
+            'date_to': invoice.date_invoice + one_day,
+        })
+
+        # Assert: The Account Code is hidden
+        self.assertNotIn(account.code, report_content)
+        self.assertIn(account.name, report_content)
+
+    def test_show_accounts_codes(self):
+        """The Account Code is shown
+        when `hide_accounts_codes` is disabled (default value).
+        """
+        # Arrange
+        invoice = self.posted_invoice
+        account = invoice.invoice_line_ids.account_id
+        # pre-condition: An Invoice is posted
+        self.assertEqual(invoice.state, 'open')
+
+        # Act: Print the Report containing the Invoice
+        one_day = timedelta(days=1)
+        report_content = self._get_report_content({
+            'account_balance_report_type': 'profit_loss',
+            'date_from': invoice.date_invoice - one_day,
+            'date_to': invoice.date_invoice + one_day,
+        })
+
+        # Assert: The Account Code is shown
+        self.assertIn(account.code, report_content)
+        self.assertIn(account.name, report_content)
diff --git a/l10n_it_account_balance_report/wizard/wizard_account_balance_report.py b/l10n_it_account_balance_report/wizard/wizard_account_balance_report.py
index d5b9c9887f7a..4196257894c9 100644
--- a/l10n_it_account_balance_report/wizard/wizard_account_balance_report.py
+++ b/l10n_it_account_balance_report/wizard/wizard_account_balance_report.py
@@ -13,6 +13,7 @@ class ReportAccountBalanceWizard(models.TransientModel):
          ('balance_sheet', "Balance Sheet")],
         string="Report Type"
     )
+    hide_accounts_codes = fields.Boolean()
 
     @api.onchange('show_partner_details')
     def onchange_show_partner_details(self):
@@ -63,6 +64,7 @@ def prepare_report_vals(self):
         return {
             'account_balance_report_type': self.account_balance_report_type,
             'trial_balance_id': trial_balance.id,
+            'hide_accounts_codes': self.hide_accounts_codes,
         }
 
     def prepare_trial_balance_vals(self):
diff --git a/l10n_it_account_balance_report/wizard/wizard_account_balance_report.xml b/l10n_it_account_balance_report/wizard/wizard_account_balance_report.xml
index 29a8bda8461f..45abd9c6a0f8 100644
--- a/l10n_it_account_balance_report/wizard/wizard_account_balance_report.xml
+++ b/l10n_it_account_balance_report/wizard/wizard_account_balance_report.xml
@@ -29,6 +29,7 @@
                                 <field name="target_move"
                                        widget="radio"/>
                                 <field name="hide_account_at_0"/>
+                                <field name="hide_accounts_codes"/>
                                 <field name="show_partner_details"/>
                                 <field name="hierarchy_on" invisible="1"/>
                                 <field name="limit_hierarchy_level"