fix errores tests

Author: zaoral

l10n_uy_currency_update/models/res_currency.py

@@ -5,6 +5,7 @@
 import logging
 _logger = logging.getLogger(__name__)
 
+
 class ResCurrency(models.Model):
 
     _inherit = "res.currency"

l10n_uy_ux/i18n/es.po

@@ -180,11 +180,6 @@ msgstr "CFE archivo xml"
 msgid "Cancel"
 msgstr "Cancelar"
 
-#. module: l10n_uy_edi
-#: model:ir.model.fields,field_description:l10n_uy_edi.field_account_move__l10n_uy_edi_cfe_uuid
-msgid "Clave o UUID del CFE"
-msgstr ""
-
 #. module: l10n_uy_edi
 #: model_terms:ir.ui.view,arch_db:l10n_uy_edi.view_base_partner_update_from_padron_uy_form
 msgid "Close"
@@ -744,15 +739,6 @@ msgstr ""
 msgid "UCFE Tipo de Notificacion"
 msgstr ""
 
-#. module: l10n_uy_edi
-#: model:ir.model.fields,help:l10n_uy_edi.field_account_move__l10n_uy_edi_cfe_uuid
-msgid ""
-"Unique identification per CFE in UCFE. Currently is formed by the "
-"concatenation of model name + record id"
-msgstr ""
-"Identificador único del CFE en el UFCE. Este esta formado por la "
-"contanación de el nombre del mdelo y el id del registro"
-
 #. module: l10n_uy_edi
 #: model_terms:ir.ui.view,arch_db:l10n_uy_edi.view_base_partner_update_from_padron_uy_form
 msgid "Update Automatically"

l10n_uy_ux/models/account_move.py

@@ -69,24 +69,6 @@ def l10n_uy_edi_action_download_preview_xml(self):
 
         return super().l10n_uy_edi_action_download_preview_xml()
 
-    def _l10n_uy_edi_cfe_A_receptor(self):
-        # EXTEND l10n_uy_edi
-        """ Agregamos campos que existen en odoo modulo oficial y queremos enviar en el xml
-        """
-        res = super()._l10n_uy_edi_cfe_A_receptor()
-        if self._is_uy_resguardo():
-            res.pop("CompraID", False)
-        return res
-
-    def _l10n_uy_edi_cfe_C_totals(self, tax_details):
-        # EXTEND l10n_uy_edi
-        """ A130 Monto Total a Pagar (NO debe ser reportado si de tipo e-resguardo) """
-        # TODO KZ mover esto a modulo e-resguardo una vez lo tengamos
-        res = super()._l10n_uy_edi_cfe_A_receptor(tax_details)
-        if self._is_uy_resguardo():
-            res.pop("MntPagar")
-        return res
-
     # New methods
 
     def uy_ux_action_get_uruware_cfe(self):
@@ -166,8 +148,6 @@ def _compute_l10n_latam_document_number(self):
                 name = name.split(" ")[-1]
             rec.l10n_latam_document_number = name
 
-    # def _l10n_uy_edi_cfe_F_reference(self):
-    #     # TODO KZ Not sure if FechaCFEref": 2015-01-31, shuould be inform
 
     # Nuevos metodos
 
@@ -285,6 +265,3 @@ def is_zona_franca(self):
 
     # TODO KZ esto lo tendriamos que mantener para nuestros clientes que tiene el nombre largo como prefijo de
     # documento. capaz lo mejor seria hacer un script para poner todo como hace Odoo. Si hacemos eso este metodo se va
-
-    # def _uy_cfe_A41_RznSoc(self):
-    # TODO company register name?

l10n_uy_ux/models/l10n_uy_edi_document.py

@@ -1,10 +1,14 @@
 import re
+import logging
+
 
 from odoo import _, api, models
 
 from odoo.exceptions import UserError
 from odoo.tools import safe_eval
 
+_logger = logging.getLogger(__name__)
+
 
 class L10nUyEdiDocument(models.Model):
 
@@ -14,8 +18,10 @@ class L10nUyEdiDocument(models.Model):
 
     def _get_ws_url(self, ws_endpoint, company):
         # EXTEND l10n_uy_edi
-        """ Si utiliza uruware por contrato externo. modificar para soportar las dos url,
-        una de testing y una de prod, asi no tiene que configurar el dato cada vez """
+        """ Si utiliza uruware por Contrato Externo (no el de Odoo) da la posibilidad
+        de utilizar dos urls en system parametros, uno para test y otro para pod
+
+        Asi no tiene que configurar el dato cada vez que lo vayan a usar """
         url = super()._get_ws_url(ws_endpoint, company)
 
         if company.l10n_uy_edi_ucfe_env == "demo":
@@ -27,26 +33,29 @@ def _get_ws_url(self, ws_endpoint, company):
         query_param = self.env["ir.config_parameter"].sudo().get_param(
                 "l10n_uy_edi.l10n_uy_edi_ucfe_query_url" + company.l10n_uy_edi_ucfe_env)
 
+        pattern = {
+            "inbox": "https://.*.ucfe.com.uy/inbox.*/cfeservice.svc",
+            "query": "https://.*.ucfe.com.uy/query.*/webservicesfe.svc",
+        }
         if ws_endpoint == "inbox" and inbox_param:
             url = inbox_param
-            pattern = "https://.*.ucfe.com.uy/inbox.*/cfeservice.svc"
         elif ws_endpoint == "query" and query_param:
             url = query_param
-            pattern = "https://.*.ucfe.com.uy/query.*/webservicesfe.svc"
+        else:
+            _logger.info("Using Odoo defaults values")
 
-        print(" ----- url %s" % url)
-        return url if re.match(pattern, url, re.IGNORECASE) is not None else False
+        return url if re.match(pattern[ws_endpoint], url, re.IGNORECASE) is not None else False
 
     def action_update_dgi_state(self):
         # EXTEND l10n_uy_edi
         """ Permitimos actualizar estado solo si tenemos UUID y solo si esta en esperando respuesta.
-        si hay error no hay nada que consultar, y si fue aceptado rechazado ya no necesita ser actualizado """
-        for move in self:
-            if not move.l10n_uy_edi_cfe_uuid:
+        Si hay error no hay nada que consultar, y si fue aceptado rechazado ya no necesita ser actualizado """
+        for doc in self:
+            if not doc.uuid:
                 raise UserError(_("Please return a 'UUID CFE Key' in order to continue"))
-            if move.l10n_uy_edi_cfe_state == "error":
+            if doc.state == "error":
                 raise UserError(_("You can not obtain the invoice with errors"))
-            if move.l10n_uy_edi_cfe_state != "received":
+            if doc.state != "received":
                 raise UserError(_("You can not update the state of a accepted/rejected invoice"))
 
         super().action_update_dgi_state()