[16.0][MIG] l10n_es_aeat_mod349: Migration to 16.0

l10n_es_aeat_mod349/__manifest__.py

@@ -9,7 +9,7 @@
 
 {
     "name": "AEAT modelo 349",
-    "version": "15.0.1.3.0",
+    "version": "16.0.1.0.0",
     "author": "Tecnativa, ForgeFlow, Odoo Community Association (OCA)",
     "license": "AGPL-3",
     "category": "Localisation/Accounting",

l10n_es_aeat_mod349/models/mod349.py

@@ -126,8 +126,8 @@ def _create_349_details(self, move_lines):
             self._create_349_record_detail(move_line)
 
     def _create_349_record_detail(self, move_line):
-        types = move_line.move_id.mapped("line_ids.account_id.internal_type")
-        sign = 1 if "payable" in types else -1
+        types = move_line.move_id.mapped("line_ids.account_id.account_type")
+        sign = 1 if "liability_payable" in types else -1
         return self.env["l10n.es.aeat.mod349.partner_record_detail"].create(
             {
                 "report_id": self.id,
@@ -137,8 +137,8 @@ def _create_349_record_detail(self, move_line):
         )
 
     def _create_349_refund_detail(self, move_line):
-        types = move_line.move_id.mapped("line_ids.account_id.internal_type")
-        sign = 1 if "receivable" in types else -1
+        types = move_line.move_id.mapped("line_ids.account_id.account_type")
+        sign = 1 if "asset_receivable" in types else -1
         return self.env["l10n.es.aeat.mod349.partner_refund_detail"].create(
             {
                 "report_id": self.id,
@@ -199,10 +199,10 @@ def _create_349_refund_records(self):
         for refund_detail in self.partner_refund_detail_ids:
             move_line = refund_detail.refund_line_id
             origin_invoice = move_line.move_id.reversed_entry_id
-            groups.setdefault(origin_invoice, refund_detail_obj)
-            groups[origin_invoice] += refund_detail
-        for origin_invoice in groups:
-            refund_details = groups[origin_invoice]
+            key = (origin_invoice, move_line.l10n_es_aeat_349_operation_key)
+            groups.setdefault(key, refund_detail_obj)
+            groups[key] += refund_detail
+        for (origin_invoice, op_key), refund_details in groups.items():
             refund_detail = first(refund_details)
             move_line = refund_detail.refund_line_id
             partner = move_line.partner_id
@@ -382,7 +382,7 @@ def calculate(self):
             self._create_349_invoice_records()
             self._create_349_refund_records()
         # Recompute all pending computed fields
-        self.recompute()
+        self.flush_recordset()
         return True
 
     def button_recover(self):
@@ -427,7 +427,7 @@ def button_confirm(self):
         """Checks if all the fields of the report are correctly filled"""
         self._check_names()
         self._check_report_lines()
-        return super(Mod349, self).button_confirm()
+        return super().button_confirm()
 
 
 class Mod349PartnerRecord(models.Model):

l10n_es_aeat_mod349/tests/test_l10n_es_aeat_mod349.py

@@ -208,5 +208,5 @@ def test_model_349(self):
             self.assertTrue(export_to_boe._export_config(model349, export_config))
         # Test report printing
         self.env.ref("l10n_es_aeat_mod349.act_report_aeat_mod349_pdf")._render(
-            model349.ids
+            "l10n_es_aeat_mod349.report_l10n_es_mod349_pdf", model349.ids, {}
         )

l10n_es_aeat_mod349/views/account_move_view.xml

@@ -3,19 +3,22 @@
     <record id="view_move_form" model="ir.ui.view">
         <field name="model">account.move</field>
         <field name="inherit_id" ref="account.view_move_form" />
-        <field name="groups_id" eval="[(4, ref('account.group_account_invoice'))]" />
         <field name="arch" type="xml">
             <field name="fiscal_position_id" position="after">
                 <field
                     name="eu_triangular_deal"
-                    groups="l10n_es_aeat.group_account_aeat"
+                    groups="l10n_es_aeat.group_account_aeat,account.group_account_invoice"
                 />
             </field>
             <xpath
                 expr="//field[@name='invoice_line_ids']/tree/field[@name='tax_ids']"
                 position="after"
             >
-                <field name="l10n_es_aeat_349_operation_key" invisible="1" />
+                <field
+                    name="l10n_es_aeat_349_operation_key"
+                    invisible="1"
+                    groups="l10n_es_aeat.group_account_aeat,account.group_account_invoice"
+                />
             </xpath>
         </field>
     </record>

setup/l10n_es_aeat_mod349/odoo/addons/l10n_es_aeat_mod349

@@ -0,0 +1 @@
+../../../../l10n_es_aeat_mod349

setup/l10n_es_aeat_mod349/setup.py

@@ -0,0 +1,6 @@
+import setuptools
+
+setuptools.setup(
+    setup_requires=['setuptools-odoo'],
+    odoo_addon=True,
+)