[MIG] l10n_es_facturae: Migration to 16.0

OCA · May 15, 2023 · b1d7b0f · b1d7b0f
1 parent 98b7f33
commit b1d7b0f
Show file tree

Hide file tree

Showing 5 changed files with 23 additions and 18 deletions.
diff --git a/l10n_es_facturae/__manifest__.py b/l10n_es_facturae/__manifest__.py
@@ -8,7 +8,7 @@
 
 {
     "name": "Creación de Facturae",
-    "version": "15.0.1.4.0",
+    "version": "16.0.1.0.0",
     "author": "ASR-OSS, "
     "FactorLibre, "
     "Tecon, "

diff --git a/l10n_es_facturae/tests/common.py b/l10n_es_facturae/tests/common.py
@@ -4,10 +4,10 @@
 
 import base64
 from datetime import timedelta
+from unittest import mock
 
 import xmlsig
 from lxml import etree
-from mock import patch
 
 from odoo import exceptions, fields
 from odoo.tools.misc import mute_logger
@@ -262,7 +262,7 @@ def test_facturae_with_attachments(self):
         self.move.action_post()
         self.move.name = "2999/99999"
         self.partner.attach_invoice_as_annex = True
-        with patch(
+        with mock.patch(
             "odoo.addons.base.models.ir_actions_report.IrActionsReport._render_qweb_pdf"
         ) as ptch:
             ptch.return_value = (b"1234", "pdf")

diff --git a/l10n_es_facturae/views/account_move_view.xml b/l10n_es_facturae/views/account_move_view.xml
@@ -28,6 +28,7 @@
                     icon="fa-tags"
                     attrs="{'column_invisible': [('parent.facturae','=', False)]}"
                     type="object"
+                    title="Edit"
                 />
             </xpath>
             <notebook position="inside">

diff --git a/l10n_es_facturae/views/res_partner_view.xml b/l10n_es_facturae/views/res_partner_view.xml
@@ -6,10 +6,9 @@
         <field name="inherit_id" ref="account.view_partner_property_form" />
         <field name="priority">99</field>
         <field name="type">form</field>
-        <field name="groups_id" eval="[(4, ref('account.group_account_invoice'))]" />
         <field name="arch" type="xml">
             <xpath expr="//field[@name='child_ids']/form/sheet/group" position="inside">
-                <group>
+                <group groups="account.group_account_invoice">
                     <group
                         string="Facturae"
                         attrs="{'invisible': ['|', ('parent.facturae', '=', False), ('type', '!=', 'invoice')]}"
@@ -24,15 +23,19 @@
                     </group>
                 </group>
             </xpath>
-            <group name='accounting_entries' position="inside">
-                <field name="facturae" />
+            <xpath expr="/form" position="inside">
                 <field name="parent_facturae" invisible="1" />
+                <field name="facturae" invisible="1" />
+            </xpath>
+            <group name='accounting_entries' position="inside">
+                <field name="facturae" groups="account.group_account_invoice" />
             </group>
             <group name="accounting_entries" position="after">
                 <group
                     name="group_facturae"
                     string="Facturae"
                     attrs="{'invisible': [('facturae', '=', False), ('parent_facturae', '=', False)]}"
+                    groups="account.group_account_invoice"
                 >
                     <field name="facturae_version" />
                     <field
@@ -56,23 +59,24 @@
                     name="group_facturae"
                     string="Facturae"
                     attrs="{'invisible': ['|', ('type', '!=', 'invoice'), ('parent_facturae', '=', False)]}"
+                    groups="account.group_account_invoice"
                 >
-                <field name="facturae_version" />
-                <field
+                    <field name="facturae_version" />
+                    <field
                         name="organo_gestor"
                         attrs="{'required': [('facturae', '=', True)]}"
                     />
-                <field
+                    <field
                         name="unidad_tramitadora"
                         attrs="{'required': [('facturae', '=', True)]}"
                     />
-                <field
+                    <field
                         name="oficina_contable"
                         attrs="{'required': [('facturae', '=', True)]}"
                     />
-                <field name="organo_proponente" />
-                <field name="attach_invoice_as_annex" />
-            </group>
+                    <field name="organo_proponente" />
+                    <field name="attach_invoice_as_annex" />
+                </group>
             </page>
         </field>
     </record>

diff --git a/l10n_es_facturae/wizard/create_facturae.py b/l10n_es_facturae/wizard/create_facturae.py
@@ -57,13 +57,13 @@ def create_facturae_file(self):
         assert active_model == "account.move", "Bad context propagation"
         move = self.env["account.move"].browse(move_ids[0]).ensure_one()
         if self.firmar_facturae:
-            move_file = self.env.ref("l10n_es_facturae.report_facturae_signed")._render(
-                move.ids
+            move_file = self.env["ir.actions.report"]._render(
+                "l10n_es_facturae.report_facturae_signed", move.ids
             )[0]
             file_name = ("facturae_" + move.name + ".xsig").replace("/", "-")
         else:
-            move_file = self.env.ref("l10n_es_facturae.report_facturae")._render(
-                move.ids
+            move_file = self.env["ir.actions.report"]._render(
+                "l10n_es_facturae.report_facturae", move.ids
             )[0]
             file_name = ("facturae_" + move.name + ".xml").replace("/", "-")
         file = base64.b64encode(move_file)