[MIG] l10n_es_aeat_mod349: Migration to 19.0

Author: sergioM-S73

l10n_es_aeat_mod349/README.rst

@@ -21,13 +21,13 @@ AEAT modelo 349
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fl10n--spain-lightgray.png?logo=github
-    :target: https://github.com/OCA/l10n-spain/tree/18.0/l10n_es_aeat_mod349
+    :target: https://github.com/OCA/l10n-spain/tree/19.0/l10n_es_aeat_mod349
     :alt: OCA/l10n-spain
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/l10n-spain-18-0/l10n-spain-18-0-l10n_es_aeat_mod349
+    :target: https://translation.odoo-community.org/projects/l10n-spain-19-0/l10n-spain-19-0-l10n_es_aeat_mod349
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
-    :target: https://runboat.odoo-community.org/builds?repo=OCA/l10n-spain&target_branch=18.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/l10n-spain&target_branch=19.0
     :alt: Try me on Runboat
 
 |badge1| |badge2| |badge3| |badge4| |badge5|
@@ -41,56 +41,56 @@ físicos y lógicos del 349.
 De acuerdo con la normativa de la Hacienda Española, están obligados a
 presentar el modelo 349:
 
-   - Todos aquellos sujetos pasivos del Impuesto sobre el Valor Añadido
-     que hayan realizado las operaciones previstas en el artículo 79 del
-     Reglamento del Impuesto sobre el Valor Añadido, es decir, quienes
-     adquieran o vendan bienes a empresas situadas en países miembros de
-     la UE, sino también aquellos que presten servicios a miembros de la
-     UE y cumplan con las siguientes condiciones:
+   -  Todos aquellos sujetos pasivos del Impuesto sobre el Valor Añadido
+      que hayan realizado las operaciones previstas en el artículo 79
+      del Reglamento del Impuesto sobre el Valor Añadido, es decir,
+      quienes adquieran o vendan bienes a empresas situadas en países
+      miembros de la UE, sino también aquellos que presten servicios a
+      miembros de la UE y cumplan con las siguientes condiciones:
 
    ..
 
-      - Que conforme a las reglas de la localización aplicables a las
-        mismas, no se entiendan prestadas en el territorio de aplicación
-        del impuesto.
-
-      - Que estén sometidas efectivamente a gravamen de otro Estado
-        miembro.
-
-      - Que su destinatario sea un empresario o profesional actuando
-        como tal y radique en dicho Estado miembro la sede de su
-        actividad económica, o tenga en el mismo un establecimiento
-        permanente o, en su defecto, el lugar de su domicilio o
-        residencia habitual, o que dicho destinatario sea una persona
-        jurídica que no actúe como empresario o profesional pero tenga
-        asignado un número de identificación a efectos del Impuesto
-        suministrado por ese Estado miembro.
-
-      - Que el sujeto pasivo sea dicho destinatario.
-
-        El período de declaración comprenderá, con carácter general las
-        operaciones realizadas en cada mes natural, y se presentará
-        durante los veinte primeros días naturales del mes inmediato
-        siguiente al correspondiente período mensual. No obstante, la
-        presentación podrá ser bimestral, trimestral o anual en los
-        siguientes supuestos:
-
-   - Bimestral: Si al final del segundo mes de un trimestre natural el
-     importe total de las entregas de bienes y prestaciones de servicios
-     que deban consignarse en la declaración recapitulativa supera
-     100.000 euros (a partir de 2012, el umbral se fija en 50.000
-     euros).
-   - Trimestral: Cuando ni durante el trimestre de referencia ni en cada
-     uno de los cuatro trimestres naturales anteriores el importe total
-     de las entregas de bienes y prestaciones de servicios que deban
-     consignarse en la declaración recapitulativa sea superior a 100.000
-     euros.
-   - Anual: En los treinta primeros días de enero del año siguiente ( la
-     primera sería en enero de 2011) si el importe total de las entregas
-     de bienes o prestaciones de servicios del año ( excluido IVA), no
-     supera los 35.000 € y el importe total de las entregas de bienes a
-     otro Estado Miembro (salvo medios de transporte nuevos) exentas de
-     IVA no sea superior a 15.000 €.
+      -  Que conforme a las reglas de la localización aplicables a las
+         mismas, no se entiendan prestadas en el territorio de
+         aplicación del impuesto.
+
+      -  Que estén sometidas efectivamente a gravamen de otro Estado
+         miembro.
+
+      -  Que su destinatario sea un empresario o profesional actuando
+         como tal y radique en dicho Estado miembro la sede de su
+         actividad económica, o tenga en el mismo un establecimiento
+         permanente o, en su defecto, el lugar de su domicilio o
+         residencia habitual, o que dicho destinatario sea una persona
+         jurídica que no actúe como empresario o profesional pero tenga
+         asignado un número de identificación a efectos del Impuesto
+         suministrado por ese Estado miembro.
+
+      -  Que el sujeto pasivo sea dicho destinatario.
+
+         El período de declaración comprenderá, con carácter general las
+         operaciones realizadas en cada mes natural, y se presentará
+         durante los veinte primeros días naturales del mes inmediato
+         siguiente al correspondiente período mensual. No obstante, la
+         presentación podrá ser bimestral, trimestral o anual en los
+         siguientes supuestos:
+
+   -  Bimestral: Si al final del segundo mes de un trimestre natural el
+      importe total de las entregas de bienes y prestaciones de
+      servicios que deban consignarse en la declaración recapitulativa
+      supera 100.000 euros (a partir de 2012, el umbral se fija en
+      50.000 euros).
+   -  Trimestral: Cuando ni durante el trimestre de referencia ni en
+      cada uno de los cuatro trimestres naturales anteriores el importe
+      total de las entregas de bienes y prestaciones de servicios que
+      deban consignarse en la declaración recapitulativa sea superior a
+      100.000 euros.
+   -  Anual: En los treinta primeros días de enero del año siguiente (
+      la primera sería en enero de 2011) si el importe total de las
+      entregas de bienes o prestaciones de servicios del año ( excluido
+      IVA), no supera los 35.000 € y el importe total de las entregas de
+      bienes a otro Estado Miembro (salvo medios de transporte nuevos)
+      exentas de IVA no sea superior a 15.000 €.
 
 **Table of contents**
 
@@ -102,36 +102,38 @@ Usage
 
 Ir a:
 
-- Contabilidad / Informes AEAT / Modelo 349
-- Crear un nuevo registro e informar los datos básicos de la
-  declaración.
-- Pulsar 'Calcular' y revisar el resultado. Pulsar el botón 'Imprimir'
-  para obtener el resultado en PDF.
-- Para excluir ciertas operaciones de la declaración, ir a las pestañas
-  'Registros de empresas' o 'Rectificaciones', y eliminar, en la seccion
-  'Detalles', las operaciones que se desee excluir de la declaración.
+-  Contabilidad / Informes AEAT / Modelo 349
+-  Crear un nuevo registro e informar los datos básicos de la
+   declaración.
+-  Pulsar 'Calcular' y revisar el resultado. Pulsar el botón 'Imprimir'
+   para obtener el resultado en PDF.
+-  Para excluir ciertas operaciones de la declaración, ir a las pestañas
+   'Registros de empresas' o 'Rectificaciones', y eliminar, en la
+   seccion 'Detalles', las operaciones que se desee excluir de la
+   declaración.
 
 Consideraciones importantes:
 
-- En caso de indicar el tipo de declaración 'Suplementaria' o 'Normal'
-  se propondrán todas las operaciones que apliquen para el periodo.
-- En caso de indicar 'Complementaria', se propondrán únicamente aquellas
-  operaciones que no hubieran sido aún presentadas en otra declaración.
+-  En caso de indicar el tipo de declaración 'Suplementaria' o 'Normal'
+   se propondrán todas las operaciones que apliquen para el periodo.
+-  En caso de indicar 'Complementaria', se propondrán únicamente
+   aquellas operaciones que no hubieran sido aún presentadas en otra
+   declaración.
 
 Known issues / Roadmap
 ======================
 
-- Las facturas rectificativas que no se hayan realizado desde otra
-  factura quedarán descolgadas y no se presentarán en el 349.
-- Añadir aclaración de cuál es el error en los registros.
+-  Las facturas rectificativas que no se hayan realizado desde otra
+   factura quedarán descolgadas y no se presentarán en el 349.
+-  Añadir aclaración de cuál es el error en los registros.
 
 Bug Tracker
 ===========
 
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/l10n-spain/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/l10n-spain/issues/new?body=module:%20l10n_es_aeat_mod349%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/l10n-spain/issues/new?body=module:%20l10n_es_aeat_mod349%0Aversion:%2019.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 
 Do not contact contributors directly about support or help with technical issues.
 
@@ -147,43 +149,43 @@ Authors
 Contributors
 ------------
 
-- Luis Manuel Angueira Blanco (Pexego)
+-  Luis Manuel Angueira Blanco (Pexego)
 
-- Omar Castiñeira Saavedra<omar@pexego.es>
+-  Omar Castiñeira Saavedra<omar@pexego.es>
 
-- Miguel López (Top Consultant)
+-  Miguel López (Top Consultant)
 
-- Ignacio Martínez (Top Consultant)
+-  Ignacio Martínez (Top Consultant)
 
-- `Tecnativa <https://www.tecnativa.com>`__:
+-  `Tecnativa <https://www.tecnativa.com>`__:
 
-  - Pedro M. Baeza
-  - Ángel Moya
-  - Luis M. Ontalba
-  - Carlos Daudén
+   -  Pedro M. Baeza
+   -  Ángel Moya
+   -  Luis M. Ontalba
+   -  Carlos Daudén
 
-- ForgeFlow (http://www.forgeflow.com)
+-  ForgeFlow (http://www.forgeflow.com)
 
-  - Jordi Ballester <jordi.ballester@forgeflow.com>
-  - Aarón Henríquez
+   -  Jordi Ballester <jordi.ballester@forgeflow.com>
+   -  Aarón Henríquez
 
-- Aitor Bouzas <aitor.bouzas@adaptivecity.com>
+-  Aitor Bouzas <aitor.bouzas@adaptivecity.com>
 
-- Acysos:
+-  Acysos:
 
-  - Ignacio Ibeas
+   -  Ignacio Ibeas
 
-- `Sygel <https://www.sygel.es>`__:
+-  `Sygel <https://www.sygel.es>`__:
 
-  - Valentin Vinagre
-  - Manuel Regidor
+   -  Valentin Vinagre
+   -  Manuel Regidor
 
-- Jairo Llopis (Moduon)
+-  Jairo Llopis (Moduon)
 
-- `Factor Libre <https://factorlibre.com>`__:
+-  `Factor Libre <https://factorlibre.com>`__:
 
-     - Luis J. Salvatierra <luis.salvatierra@factorlibre.com>
-     - Alejandro Ji Cheung <alejandro.jicheung@factorlibre.com>
+      -  Luis J. Salvatierra <luis.salvatierra@factorlibre.com>
+      -  Alejandro Ji Cheung <alejandro.jicheung@factorlibre.com>
 
 Maintainers
 -----------
@@ -206,6 +208,6 @@ Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
 
 |maintainer-pedrobaeza| 
 
-This module is part of the `OCA/l10n-spain <https://github.com/OCA/l10n-spain/tree/18.0/l10n_es_aeat_mod349>`_ project on GitHub.
+This module is part of the `OCA/l10n-spain <https://github.com/OCA/l10n-spain/tree/19.0/l10n_es_aeat_mod349>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

l10n_es_aeat_mod349/__manifest__.py

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

l10n_es_aeat_mod349/models/aeat_349_map_line.py

@@ -10,13 +10,10 @@ class Aeat349MapLines(models.Model):
     _description = "Aeat 349 Map Line"
     _rec_name = "operation_key"
 
-    _sql_constraints = [
-        (
-            "unique_operation_key",
-            "UNIQUE(operation_key)",
-            "There's already another record with the same operation key",
-        ),
-    ]
+    _unique_operation_key = models.Constraint(
+        "UNIQUE(operation_key)",
+        "There's already another record with the same operation key",
+    )
 
     def _selection_operation_key(self):
         return self.env["account.move.line"].fields_get(

l10n_es_aeat_mod349/models/mod349.py

@@ -11,7 +11,6 @@
 import math
 
 from odoo import api, exceptions, fields, models
-from odoo.fields import first
 from odoo.tools import float_is_zero, float_round
 
 
@@ -73,7 +72,7 @@ def _compute_error_count(self):
         ret_val = super()._compute_error_count()
         partner_records_error_dict = self.env[
             "l10n.es.aeat.mod349.partner_record"
-        ].read_group(
+        ]._read_group(
             domain=[("partner_record_ok", "=", False), ("report_id", "in", self.ids)],
             fields=["report_id"],
             groupby=["report_id"],
@@ -201,7 +200,7 @@ def _create_349_refund_records(self):
             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)
+            refund_detail = next(iter(refund_details), None)
             move_line = refund_detail.refund_line_id
             partner = move_line.partner_id
             op_key = move_line.l10n_es_aeat_349_operation_key
@@ -274,7 +273,7 @@ def _create_349_refund_records(self):
                 if original_amls:
                     original_move = original_amls[:1]
                     year = str(original_move.date.year)
-                    month = "%02d" % (original_move.date.month)
+                    month = f"{original_move.date.month:02d}"
                 else:
                     continue  # We can't find information to attach to
                 if self.period_type == "0A":