[FIX] account_edi_ubl_cii: check validaty of taxes

Invalid taxes could lead to invalid results from the
`_prepare_edi_tax_details` fonction. For instance: a tax without tax
repartition lines will be disregarded, which in turns leads to an
invalid xml being generated.

opw-3175701

closes odoo/odoo#116359

Signed-off-by: Nicolas Viseur (vin) <vin@odoo.com>

addons/account_edi_ubl_cii/i18n/account_edi_ubl_cii.pot

@@ -251,6 +251,12 @@ msgstr ""
 msgid "Report Action"
 msgstr ""
 
+#. module: account_edi_ubl_cii
+#: code:addons/account_edi_ubl_cii/models/account_edi_common.py:0
+#, python-format
+msgid "Tax '%s' is invalid: %s"
+msgstr ""
+
 #. module: account_edi_ubl_cii
 #: model:ir.model,name:account_edi_ubl_cii.model_account_edi_xml_ubl_nl
 msgid "SI-UBL 2.0 (NLCIUS)"

addons/account_edi_ubl_cii/models/account_edi_common.py

@@ -111,6 +111,16 @@ class AccountEdiCommon(models.AbstractModel):
     # TAXES
     # -------------------------------------------------------------------------
 
+    def _validate_taxes(self, invoice):
+        """ Validate the structure of the tax repartition lines (invalid structure could lead to unexpected results)
+        """
+        for tax in invoice.invoice_line_ids.tax_ids:
+            try:
+                tax._validate_repartition_lines()
+            except ValidationError as e:
+                error_msg = _("Tax '%s' is invalid: %s", tax.name, e.args[0])  # args[0] gives the error message
+                raise ValidationError(error_msg)
+
     def _get_tax_unece_codes(self, invoice, tax):
         """
         Source: doc of Peppol (but the CEF norm is also used by factur-x, yet not detailed)

addons/account_edi_ubl_cii/models/account_edi_xml_cii_facturx.py

@@ -123,6 +123,9 @@ class AccountEdiXmlCII(models.AbstractModel):
             # Facturx requires the monetary values to be rounded to 2 decimal values
             return float_repr(number, decimal_places)
 
+        # Validate the structure of the taxes
+        self._validate_taxes(invoice)
+
         # Create file content.
         tax_details = invoice._prepare_edi_tax_details(
             grouping_key_generator=lambda tax_values: {

addons/account_edi_ubl_cii/models/account_edi_xml_ubl_20.py

@@ -347,6 +347,9 @@ class AccountEdiXmlUBL20(models.AbstractModel):
                 '_tax_category_vals_': tax_category_vals,
             }
 
+        # Validate the structure of the taxes
+        self._validate_taxes(invoice)
+
         # Compute the tax details for the whole invoice and each invoice line separately.
         taxes_vals = invoice._prepare_edi_tax_details(grouping_key_generator=grouping_key_generator)