From 0e3b4d29722b9018b58d611b16af945279790265 Mon Sep 17 00:00:00 2001
From: valen <valentin.vinagre@sygel.es>
Date: Mon, 14 Jun 2021 21:40:04 +0200
Subject: [PATCH] [IMP] account_financial_report: added 'auto sequence' option
 in Journal Ledger

---
 .../readme/CONTRIBUTORS.rst                   |  5 +++
 .../report/journal_ledger.py                  |  8 +++-
 .../report/journal_ledger_xlsx.py             |  6 +++
 .../report/templates/journal_ledger.xml       | 39 +++++++++++++++++--
 .../wizard/journal_ledger_wizard.py           |  2 +
 .../wizard/journal_ledger_wizard_view.xml     |  1 +
 6 files changed, 56 insertions(+), 5 deletions(-)

diff --git a/account_financial_report/readme/CONTRIBUTORS.rst b/account_financial_report/readme/CONTRIBUTORS.rst
index e8ee8657..7d3a130a 100644
--- a/account_financial_report/readme/CONTRIBUTORS.rst
+++ b/account_financial_report/readme/CONTRIBUTORS.rst
@@ -24,5 +24,10 @@
   * João Marques
   * Alexandre D. Díaz
 
+* `Sygel <https://www.sygel.es>`__:
+
+  * Harald Panten
+  * Valentin Vinagre
+
 Much of the work in this module was done at a sprint in Sorrento, Italy in
 April 2016.
diff --git a/account_financial_report/report/journal_ledger.py b/account_financial_report/report/journal_ledger.py
index 2ae97810..a79a6cd5 100644
--- a/account_financial_report/report/journal_ledger.py
+++ b/account_financial_report/report/journal_ledger.py
@@ -84,7 +84,7 @@ class JournalLedgerReport(models.AbstractModel):
     def _get_move_lines_order(self, move_ids, wizard, journal_ids):
         return ""
 
-    def _get_move_lines_data(self, ml, wizard, ml_taxes):
+    def _get_move_lines_data(self, ml, wizard, ml_taxes, auto_sequence):
         base_debit = (
             base_credit
         ) = tax_debit = tax_credit = base_balance = tax_balance = 0.0
@@ -116,6 +116,7 @@ class JournalLedgerReport(models.AbstractModel):
             "tax_credit": tax_credit,
             "base_balance": base_balance,
             "tax_balance": tax_balance,
+            "auto_sequence": str(auto_sequence).zfill(6),
         }
 
     def _get_account_data(self, accounts):
@@ -200,6 +201,7 @@ class JournalLedgerReport(models.AbstractModel):
         partners = self.env["res.partner"]
         currencies = self.env["res.currency"]
         tax_lines = self.env["account.tax"]
+        auto_sequence = len(move_ids)
         for ml in move_lines:
             if ml.account_id not in accounts:
                 accounts |= ml.account_id
@@ -211,13 +213,14 @@ class JournalLedgerReport(models.AbstractModel):
                 tax_lines |= ml.tax_line_id
             if ml.move_id.id not in Move_Lines.keys():
                 Move_Lines[ml.move_id.id] = []
+                auto_sequence -= 1
             taxes = (
                 ml.id in move_line_ids_taxes_data.keys()
                 and move_line_ids_taxes_data[ml.id]
                 or {}
             )
             Move_Lines[ml.move_id.id].append(
-                self._get_move_lines_data(ml, wizard, taxes)
+                self._get_move_lines_data(ml, wizard, taxes, auto_sequence)
             )
         account_ids_data = self._get_account_data(accounts)
         partner_ids_data = self._get_partner_data(partners)
@@ -348,6 +351,7 @@ class JournalLedgerReport(models.AbstractModel):
             "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,
diff --git a/account_financial_report/report/journal_ledger_xlsx.py b/account_financial_report/report/journal_ledger_xlsx.py
index b9143389..4a886b70 100644
--- a/account_financial_report/report/journal_ledger_xlsx.py
+++ b/account_financial_report/report/journal_ledger_xlsx.py
@@ -28,6 +28,11 @@ class JournalLedgerXslx(models.AbstractModel):
             {"header": _("Account"), "field": "account_code", "width": 9},
         ]
 
+        if report.with_auto_sequence:
+            columns.insert(
+                0, {"header": _("Sequence"), "field": "auto_sequence", "width": 10}
+            )
+
         if report.with_account_name:
             columns.append(
                 {"header": _("Account Name"), "field": "account_name", "width": 15}
@@ -232,6 +237,7 @@ class JournalLedgerXslx(models.AbstractModel):
                 line["partner"] = self._get_partner_name(
                     line["partner_id"], partner_ids_data
                 )
+                line["auto_sequence"] = line["auto_sequence"]
                 line["account_code"] = account_code
                 line["account_name"] = account_name
                 line["currency_name"] = currency_name
diff --git a/account_financial_report/report/templates/journal_ledger.xml b/account_financial_report/report/templates/journal_ledger.xml
index 4ba80aa7..c6b90307 100644
--- a/account_financial_report/report/templates/journal_ledger.xml
+++ b/account_financial_report/report/templates/journal_ledger.xml
@@ -12,6 +12,7 @@
         </t>
     </template>
     <template id="report_journal_ledger_base">
+        <t t-set="with_auto_sequence" t-value="with_auto_sequence" />
         <t t-set="display_currency" t-value="foreign_currency" />
         <t t-set="display_account_name" t-value="with_account_name" />
         <t t-set="title">
@@ -92,16 +93,44 @@
     <template id="account_financial_report.report_journal_ledger_journal_table_header">
         <t t-if="not display_account_name">
             <t t-set="account_column_style">width: 8.11%;</t>
-            <t t-set="label_column_style">width: 38.92%;</t>
+            <t t-if="not with_auto_sequence">
+                <t t-set="label_column_style">
+                    width: 38.92%;
+                </t>
+            </t>
+            <t t-else="">
+                <t t-set="label_column_style">
+                    width: 31.35%;
+                </t>
+            </t>
         </t>
         <t t-else="">
-            <t t-set="account_column_style">width: 23.78%;</t>
+           <t t-if="not with_auto_sequence">
+                <t t-set="account_column_style">
+                    width: 23.78%;
+                </t>
+            </t>
+            <t t-else="">
+                <t t-set="account_column_style">
+                    width: 16.21%;
+                </t>
+            </t>
             <t t-set="label_column_style">width: 23.24%;</t>
         </t>
         <div class="act_as_thead">
             <div class="act_as_row labels">
+                <t t-if="with_auto_sequence">
+                    <div
+                        class="act_as_cell first_column"
+                        name="entry"
+                        style="width: 7.57%;"
+                    >
+                        Sequence
+                    </div>
+                </t>
                 <div
-                    class="act_as_cell first_column"
+                    t-att-class="'act_as_cell' if with_auto_sequence else 'act_as_cell first_column'"
+                    class="act_as_cell"
                     name="entry"
                     style="width: 7.57%;"
                 >
@@ -143,6 +172,7 @@
     </template>
     <template id="account_financial_report.report_journal_ledger_journal_first_line">
         <div class="act_as_row lines">
+            <div class="act_as_cell" name="Sequence" />
             <div class="act_as_cell" name="entry" />
             <div class="act_as_cell" name="date" />
             <div class="act_as_cell" name="account" />
@@ -194,6 +224,9 @@
         </t>
     </template>
     <template id="account_financial_report.report_journal_move_line">
+        <div class="act_as_cell left" name="auto_sequence" t-if="with_auto_sequence">
+            <span t-if="display_move_info" t-esc="move_line['auto_sequence']" />
+        </div>
         <div class="act_as_cell left" name="entry">
             <t t-if="display_move_info">
                 <span
diff --git a/account_financial_report/wizard/journal_ledger_wizard.py b/account_financial_report/wizard/journal_ledger_wizard.py
index 6e9e7f56..94d6a8bc 100644
--- a/account_financial_report/wizard/journal_ledger_wizard.py
+++ b/account_financial_report/wizard/journal_ledger_wizard.py
@@ -40,6 +40,7 @@ class JournalLedgerReportWizard(models.TransientModel):
         required=True,
     )
     with_account_name = fields.Boolean(default=False)
+    with_auto_sequence = fields.Boolean(string="Show Auto Sequence", default=False)
 
     @api.model
     def _get_move_targets(self):
@@ -128,6 +129,7 @@ class JournalLedgerReportWizard(models.TransientModel):
             "group_option": self.group_option,
             "with_account_name": self.with_account_name,
             "account_financial_report_lang": self.env.lang,
+            "with_auto_sequence": self.with_auto_sequence,
         }
 
     def _export(self, report_type):
diff --git a/account_financial_report/wizard/journal_ledger_wizard_view.xml b/account_financial_report/wizard/journal_ledger_wizard_view.xml
index a578501e..53f7097d 100644
--- a/account_financial_report/wizard/journal_ledger_wizard_view.xml
+++ b/account_financial_report/wizard/journal_ledger_wizard_view.xml
@@ -31,6 +31,7 @@
                         <field name="group_option" />
                         <field name="foreign_currency" />
                         <field name="with_account_name" />
+                        <field name="with_auto_sequence" />
                     </group>
                     <group />
                 </group>