[MIG] l10n_es_pos_sii: Migration to 17.0

Author: BernatObrador

l10n_es_pos_sii/README.rst

@@ -17,10 +17,10 @@ Envío de pedidos del TPV al SII
     :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/17.0/l10n_es_pos_sii
+    :target: https://github.com/OCA/l10n-spain/tree/17.0/l10n_es_pos_oca_sii
     :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-17-0/l10n-spain-17-0-l10n_es_pos_sii
+    :target: https://translation.odoo-community.org/projects/l10n-spain-17-0/l10n-spain-17-0-l10n_es_pos_oca_sii
     :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=17.0
@@ -39,7 +39,7 @@ Configuration
 =============
 
 No se requieren pasos adicionales de configuración, ver configuración de
-los módulos l10n_es_pos, l10n_es_aeat_sii_oca y pos_default_partner.
+los módulos l10n_es_pos_oca, l10n_es_aeat_sii_oca y pos_default_partner.
 
 Known issues / Roadmap
 ======================
@@ -62,7 +62,7 @@ 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_pos_sii%0Aversion:%2017.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_pos_oca_sii%0Aversion:%2017.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.
 
@@ -82,6 +82,10 @@ Contributors
    -  Almudena de la Puente <[email protected]>
    -  Jose Zambudio <[email protected]>
 
+-  `APSL-Nagarro <https://apsl.tech>`__:
+
+   -  Bernat Obrador <[email protected]>
+
 Maintainers
 -----------
 
@@ -95,6 +99,6 @@ OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.
 
-This module is part of the `OCA/l10n-spain <https://github.com/OCA/l10n-spain/tree/17.0/l10n_es_pos_sii>`_ project on GitHub.
+This module is part of the `OCA/l10n-spain <https://github.com/OCA/l10n-spain/tree/17.0/l10n_es_pos_oca_sii>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

l10n_es_pos_sii/__manifest__.py

@@ -6,20 +6,19 @@
     "author": "Aures Tic, Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/l10n-spain",
     "license": "AGPL-3",
-    "version": "16.0.2.1.1",
+    "version": "17.0.1.0.0",
     "depends": [
         "point_of_sale",
-        "l10n_es_pos",
+        "l10n_es_pos_oca",
         "l10n_es_aeat_sii_oca",
-        "pos_default_partner",
     ],
     "data": [
-        "data/aeat_sii_queue_job.xml",
+        "data/ir_cron.xml",
         "views/pos_order.xml",
         "views/res_company.xml",
     ],
     "assets": {
-        "point_of_sale.assets": [
+        "point_of_sale._assets_pos": [
             "l10n_es_pos_sii/static/src/js/**/*.js",
         ],
     },

l10n_es_pos_sii/data/aeat_sii_queue_job.xml

@@ -0,0 +1,12 @@
+<?xml version="1.0" encoding="UTF-8" ?>
+<odoo noupdate="1">
+    <record forcecreate="True" id="pos_order_send_to_sii" model="ir.cron">
+        <field name="name">Send Invoices to SII (POS)</field>
+        <field name="model_id" ref="model_pos_order" />
+        <field name="state">code</field>
+        <field name="code">model._send_to_sii()</field>
+        <field name='interval_number'>5</field>
+        <field name='interval_type'>minutes</field>
+        <field name="numbercall">-1</field>
+    </record>
+</odoo>

l10n_es_pos_sii/data/ir_cron.xml

@@ -12,15 +12,6 @@ class PosOrder(models.Model):
     _name = "pos.order"
     _inherit = ["pos.order", "sii.mixin"]
 
-    order_jobs_ids = fields.Many2many(
-        comodel_name="queue.job",
-        column1="pos_order_id",
-        column2="job_id",
-        relation="pos_order_queue_job_rel",
-        string="Connector Jobs",
-        copy=False,
-    )
-
     @api.depends("company_id", "state")
     def _compute_sii_description(self):
         for order in self:
@@ -75,19 +66,14 @@ def _is_sii_type_breakdown_required(self, taxes_dict):
         """
         return False
 
-    def _get_sii_jobs_field_name(self):
-        return "order_jobs_ids"
-
     def _get_valid_document_states(self):
         return SII_VALID_POS_ORDER_STATES
 
     def _aeat_get_partner(self):
-        partner = self.session_id.config_id.default_partner_id
+        partner = self.env.ref('l10n_es.partner_simplified', raise_if_not_found=False)
         if not partner:
             raise UserError(
-                _("You must define a default partner for POS {}").format(
-                    self.session_id.config_id.display_name,
-                )
+                _("Simplified Invoice Partner not found. Please update l10n_es module.")
             )
         return partner
 
@@ -171,3 +157,20 @@ def _get_aeat_invoice_dict_out(self, cancel=False):
         if self.amount_total < 0.0:
             inv_dict["FacturaExpedida"]["TipoRectificativa"] = "I"
         return inv_dict
+
+    @api.model
+    def _send_to_sii_valid(self):
+        documents = self.search(
+            [
+                ("state", "in", self._get_valid_document_states()),
+                ("aeat_state", "not in", ["sent", "cancelled"]),
+                ("sii_enabled", "=", True),
+                ("sii_send_date", "<=", fields.Datetime.now()),
+            ]
+        )
+        if documents:
+            documents.confirm_one_document()
+
+    @api.model
+    def _send_to_sii(self):
+        self._send_to_sii_valid()

l10n_es_pos_sii/models/pos_order.py

@@ -1,2 +1,2 @@
 No se requieren pasos adicionales de configuración, ver configuración de
-los módulos l10n_es_pos, l10n_es_aeat_sii_oca y pos_default_partner.
+los módulos l10n_es_pos_oca, l10n_es_aeat_sii_oca y pos_default_partner.

l10n_es_pos_sii/readme/CONFIGURE.md

@@ -1,3 +1,5 @@
 - [Aures Tic](https://aurestic.es):
   - Almudena de la Puente \<<[email protected]>\>
   - Jose Zambudio \<<[email protected]>\>
+- [APSL-Nagarro](<https://apsl.tech>):
+  - Bernat Obrador \<<[email protected]>\>

l10n_es_pos_sii/readme/CONTRIBUTORS.md

@@ -369,7 +369,7 @@ <h1 class="title">Envío de pedidos del TPV al SII</h1>
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! source digest: sha256:be4fd959e6a7dda853164de82e35a74a964042a5233e6b5ecb0ed8c33b35a7ba
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/l10n-spain/tree/17.0/l10n_es_pos_sii"><img alt="OCA/l10n-spain" src="https://img.shields.io/badge/github-OCA%2Fl10n--spain-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/l10n-spain-17-0/l10n-spain-17-0-l10n_es_pos_sii"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/l10n-spain&amp;target_branch=17.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/l10n-spain/tree/17.0/l10n_es_pos_oca_sii"><img alt="OCA/l10n-spain" src="https://img.shields.io/badge/github-OCA%2Fl10n--spain-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/l10n-spain-17-0/l10n-spain-17-0-l10n_es_pos_oca_sii"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/l10n-spain&amp;target_branch=17.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>Envío al SII de pedidos del TPV de forma individual.</p>
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
@@ -388,7 +388,7 @@ <h1 class="title">Envío de pedidos del TPV al SII</h1>
 <div class="section" id="configuration">
 <h1><a class="toc-backref" href="#toc-entry-1">Configuration</a></h1>
 <p>No se requieren pasos adicionales de configuración, ver configuración de
-los módulos l10n_es_pos, l10n_es_aeat_sii_oca y pos_default_partner.</p>
+los módulos l10n_es_pos_oca, l10n_es_aeat_sii_oca y pos_default_partner.</p>
 </div>
 <div class="section" id="known-issues-roadmap">
 <h1><a class="toc-backref" href="#toc-entry-2">Known issues / Roadmap</a></h1>
@@ -411,7 +411,7 @@ <h1><a class="toc-backref" href="#toc-entry-3">Bug Tracker</a></h1>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/l10n-spain/issues">GitHub Issues</a>.
 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
-<a class="reference external" href="https://github.com/OCA/l10n-spain/issues/new?body=module:%20l10n_es_pos_sii%0Aversion:%2017.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/l10n-spain/issues/new?body=module:%20l10n_es_pos_oca_sii%0Aversion:%2017.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
@@ -430,6 +430,10 @@ <h2><a class="toc-backref" href="#toc-entry-6">Contributors</a></h2>
 <li>Jose Zambudio &lt;<a class="reference external" href="mailto:jose&#64;aurestic.es">jose&#64;aurestic.es</a>&gt;</li>
 </ul>
 </li>
+<li><a class="reference external" href="https://apsl.tech">APSL-Nagarro</a>:<ul>
+<li>Bernat Obrador &lt;<a class="reference external" href="mailto:bobrador&#64;apsl.net">bobrador&#64;apsl.net</a>&gt;</li>
+</ul>
+</li>
 </ul>
 </div>
 <div class="section" id="maintainers">
@@ -441,7 +445,7 @@ <h2><a class="toc-backref" href="#toc-entry-7">Maintainers</a></h2>
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/l10n-spain/tree/17.0/l10n_es_pos_sii">OCA/l10n-spain</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/l10n-spain/tree/17.0/l10n_es_pos_oca_sii">OCA/l10n-spain</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>

l10n_es_pos_sii/static/description/index.html

@@ -0,0 +1,44 @@
+/** @odoo-module **/
+
+import {ErrorPopup} from "@point_of_sale/app/errors/popups/error_popup";
+import {InvoiceButton} from "@point_of_sale/app/screens/ticket_screen/invoice_button/invoice_button";
+import {_t} from "@web/core/l10n/translation";
+import {patch} from "@web/core/utils/patch";
+import {useService} from "@web/core/utils/hooks";
+
+patch(InvoiceButton.prototype, {
+    setup() {
+        super.setup(...arguments);
+        this.popup = useService("popup");
+    },
+
+    get commandName() {
+        let cName = super.commandName;
+        const order = this.props?.order;
+        if (order) {
+            cName = this.isAlreadyInvoiced
+                ? _t("Reprint Invoice")
+                : order.siiSessionClosed
+                ? _t("Cannot Invoice")
+                : _t("Invoice");
+        }
+        return cName;
+    },
+
+    async _invoiceOrder() {
+        const order = this.props?.order;
+        if (!order) {
+            return;
+        }
+
+        if (order.siiSessionClosed) {
+            this.popup.add(ErrorPopup, {
+                title: _t("Session is closed"),
+                body: _t("Cannot invoice order from closed session."),
+            });
+            return;
+        }
+
+        return await super._invoiceOrder(...arguments);
+    },
+});