[17.0][MIG] account_financial_report

account_financial_report/__manifest__.py

@@ -6,7 +6,7 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 {
     "name": "Account Financial Reports",
-    "version": "16.0.1.4.0",
+    "version": "17.0.1.0.0",
     "category": "Reporting",
     "summary": "OCA Financial Reports",
     "author": "Camptocamp SA,"

account_financial_report/report/general_ledger.py

@@ -209,7 +209,6 @@ def _prepare_gen_ld_data_group_partners(self, data, domain, grouped_by):
                 else:
                     prt_id = gl["partner_id"][0]
                     prt_name = gl["partner_id"][1]
-                    prt_name = prt_name._value
                 acc_id = gl["account_id"][0]
                 data[acc_id][prt_id] = self._prepare_gen_ld_data_item(gl)
                 data[acc_id][prt_id]["id"] = prt_id
@@ -236,7 +235,6 @@ def _prepare_gen_ld_data_group_taxes(self, data, domain, grouped_by):
                 if "tax_line_id" in gl and gl["tax_line_id"]:
                     tax_id = gl["tax_line_id"][0]
                     tax_name = gl["tax_line_id"][1]
-                    tax_name = tax_name._value
                 else:
                     tax_id = 0
                     tax_name = "Missing Tax"

account_financial_report/report/open_items_xlsx.py

@@ -211,6 +211,6 @@ def write_ending_balance_from_dict(
             name = my_object["code"] + " - " + my_object["name"]
             my_object["residual"] = total_amount[account_id]["residual"]
             label = _("Ending balance")
-        return super(OpenItemsXslx, self).write_ending_balance_from_dict(
+        return super().write_ending_balance_from_dict(
             my_object, name, label, report_data
         )

account_financial_report/report/trial_balance_xlsx.py

@@ -257,7 +257,7 @@ def write_line(self, line_object, type_object, report_data):
             line_object.currency_id = line_object.report_account_id.currency_id
         elif type_object == "account":
             line_object.currency_id = line_object.currency_id
-        return super(TrialBalanceXslx, self).write_line(line_object, report_data)
+        return super().write_line(line_object, report_data)
 
     def write_account_footer(self, account, name_value, report_data):
         """Specific function to write account footer for Trial Balance"""

account_financial_report/static/src/js/report_action.esm.js

@@ -1,40 +1,40 @@
-/** @odoo-module **/
-import {ReportAction} from "@web/webclient/actions/reports/report_action";
-import {patch} from "web.utils";
-import {useEnrichWithActionLinks} from "./report.esm";
+// /** @odoo-module **/
+// import {ReportAction} from "@web/webclient/actions/reports/report_action";
+// import { patch } from "@web/core/utils/patch";
+// import {useEnrichWithActionLinks} from "./report.esm";
 
-const MODULE_NAME = "account_financial_report";
+// const MODULE_NAME = "account_financial_report";
 
-patch(ReportAction.prototype, "account_financial_report.ReportAction", {
-    setup() {
-        this._super.apply(this, arguments);
-        this.isAccountFinancialReport = this.props.report_name.startsWith(
-            `${MODULE_NAME}.`
-        );
-        useEnrichWithActionLinks(this.iframe);
-    },
+// patch(ReportAction.prototype, "account_financial_report.ReportAction", {
+//     setup() {
+//         this._super.apply(this, arguments);
+//         this.isAccountFinancialReport = this.props.report_name.startsWith(
+//             `${MODULE_NAME}.`
+//         );
+//         useEnrichWithActionLinks(this.iframe);
+//     },
 
-    export() {
-        this.action.doAction({
-            type: "ir.actions.report",
-            report_type: "xlsx",
-            report_name: this._get_xlsx_name(this.props.report_name),
-            report_file: this._get_xlsx_name(this.props.report_file),
-            data: this.props.data || {},
-            context: this.props.context || {},
-            display_name: this.title,
-        });
-    },
+//     export() {
+//         this.action.doAction({
+//             type: "ir.actions.report",
+//             report_type: "xlsx",
+//             report_name: this._get_xlsx_name(this.props.report_name),
+//             report_file: this._get_xlsx_name(this.props.report_file),
+//             data: this.props.data || {},
+//             context: this.props.context || {},
+//             display_name: this.title,
+//         });
+//     },
 
-    /**
-     * @param {String} str
-     * @returns {String}
-     */
-    _get_xlsx_name(str) {
-        if (!_.isString(str)) {
-            return str;
-        }
-        const parts = str.split(".");
-        return `a_f_r.report_${parts[parts.length - 1]}_xlsx`;
-    },
-});
+//     /**
+//      * @param {String} str
+//      * @returns {String}
+//      */
+//     _get_xlsx_name(str) {
+//         if (!_.isString(str)) {
+//             return str;
+//         }
+//         const parts = str.split(".");
+//         return `a_f_r.report_${parts[parts.length - 1]}_xlsx`;
+//     },
+// });

account_financial_report/static/src/xml/report.xml

@@ -4,7 +4,6 @@
         t-name="info.ReportAction"
         t-inherit="web.ReportAction"
         t-inherit-mode="extension"
-        owl="1"
     >
         <xpath expr="//button" position="after">
             <button

account_financial_report/wizard/general_ledger_wizard_view.xml

@@ -13,9 +13,7 @@
                         groups="base.group_multi_company"
                     />
                 </group>
-                <div
-                    attrs="{'invisible': [('not_only_one_unaffected_earnings_account', '=', True)]}"
-                >
+                <div invisible="not_only_one_unaffected_earnings_account == True">
                     <group name="filters">
                         <group name="date_range">
                             <field name="date_range_id" />
@@ -95,9 +93,7 @@
                         </page>
                     </notebook>
                 </div>
-                <div
-                    attrs="{'invisible': [('not_only_one_unaffected_earnings_account', '=', False)]}"
-                >
+                <div invisible="not_only_one_unaffected_earnings_account == False">
                     <field
                         name="not_only_one_unaffected_earnings_account"
                         invisible="1"
@@ -110,9 +106,7 @@
                     <group />
                 </div>
                 <footer>
-                    <div
-                        attrs="{'invisible': [('not_only_one_unaffected_earnings_account', '=', True)]}"
-                    >
+                    <div invisible="not_only_one_unaffected_earnings_account == True">
                         <button
                             name="button_export_html"
                             string="View"
@@ -135,9 +129,7 @@
                         or
                         <button string="Cancel" class="oe_link" special="cancel" />
                     </div>
-                    <div
-                        attrs="{'invisible': [('not_only_one_unaffected_earnings_account', '=', False)]}"
-                    >
+                    <div invisible="not_only_one_unaffected_earnings_account == False">
                         <button string="Cancel" class="oe_link" special="cancel" />
                     </div>
                 </footer>

account_financial_report/wizard/trial_balance_wizard_view.xml

@@ -13,9 +13,7 @@
                         groups="base.group_multi_company"
                     />
                 </group>
-                <div
-                    attrs="{'invisible': [('not_only_one_unaffected_earnings_account', '=', True)]}"
-                >
+                <div invisible="not_only_one_unaffected_earnings_account == True">
                     <group name="filters">
                         <group name="date_range">
                             <field name="date_range_id" />
@@ -29,26 +27,26 @@
                             <field name="show_partner_details" />
                             <field
                                 name="show_hierarchy"
-                                attrs="{'invisible':[('show_partner_details','=',True)]}"
+                                invisible="show_partner_details == True"
                             />
                             <field
                                 name="limit_hierarchy_level"
-                                attrs="{'invisible':['|', ('show_hierarchy','=',False),('show_partner_details','=',True)]}"
+                                invisible="show_hierarchy == False or show_partner_details == True"
                             />
                             <field
                                 name="show_hierarchy_level"
-                                attrs="{'invisible':[('limit_hierarchy_level','=', False)]}"
+                                invisible="limit_hierarchy_level == False"
                             />
                             <field
                                 name="hide_parent_hierarchy_level"
-                                attrs="{'invisible':[('limit_hierarchy_level','=', False)]}"
+                                invisible="limit_hierarchy_level == False"
                             />
                             <field name="foreign_currency" />
                         </group>
                     </group>
                     <group
                         name="partner_filter"
-                        attrs="{'invisible':[('show_partner_details','!=',True)]}"
+                        invisible="show_partner_details == True"
                         col="1"
                     >
                         <label for="partner_ids" />
@@ -66,7 +64,7 @@
                         nolabel="1"
                         options="{'no_create': True}"
                     />
-                    <group attrs="{'invisible':[('show_partner_details','!=',True)]}" />
+                    <group invisible="show_partner_details == True" />
                     <div />
                     <group name="account_filter" col="4">
                         <label for="account_ids" colspan="4" />
@@ -97,9 +95,7 @@
                         />
                     </group>
                 </div>
-                <div
-                    attrs="{'invisible': [('not_only_one_unaffected_earnings_account', '=', False)]}"
-                >
+                <div invisible="not_only_one_unaffected_earnings_account == False">
                     <field
                         name="not_only_one_unaffected_earnings_account"
                         invisible="1"
@@ -112,9 +108,7 @@
                     <group />
                 </div>
                 <footer>
-                    <div
-                        attrs="{'invisible': [('not_only_one_unaffected_earnings_account', '=', True)]}"
-                    >
+                    <div invisible="not_only_one_unaffected_earnings_account == True">
                         <button
                             name="button_export_html"
                             string="View"
@@ -137,9 +131,7 @@
                         or
                         <button string="Cancel" class="oe_link" special="cancel" />
                     </div>
-                    <div
-                        attrs="{'invisible': [('not_only_one_unaffected_earnings_account', '=', False)]}"
-                    >
+                    <div invisible="not_only_one_unaffected_earnings_account == False">
                         <button string="Cancel" class="oe_link" special="cancel" />
                     </div>
                 </footer>