Skip to content

GitLab

Merged
Opened by Cuauhtémoc Díaz Minor@cuauhtemoc.diazContributor

fix(params.pac): se cachan las excepciones de suds

All threads resolved
Compare
and
1 file
+ 75
- 52
l10n_mx_params_pac/pacs/finfok.py
+ 75
- 52
Show all unchanged lines Show 20 lines
import logging
from suds.client import Client
from suds.client import (
WebFault,
MethodNotFound,
PortNotFound,
ServiceNotFound,
TypeNotFound,
BuildError,
SoapHeadersNotPermitted,
) # noqa
from openerp import api, fields, models
from openerp.exceptions import Warning as UserError
Show 20 lines Show all unchanged lines Show 20 lines
# that's why I created a dictionary with all errors
# described in its manual
_cancel_errors = {
'300': 'Usuario y contraseña inválidos',
'202': 'Esta intentando cancelar una factura que ya ha sido cancelada',
'203': 'No corresponde el RFC del Emisor y de quien solicita '
'la cancelación',
'205': 'UUID No existe',
'704': 'Error con la contraseña de la llave Privada',
'708': 'No se pudo conectar al SAT',
'711': 'Error con el certificado al cancelar',
'712': 'El número de noCertificado es diferente al del número de'
'certificado del atributo certificado',
"300": "Usuario y contraseña inválidos",
"202": "Esta intentando cancelar una factura que ya ha sido cancelada",
"203": "No corresponde el RFC del Emisor y de quien solicita " "la cancelación",
"205": "UUID No existe",
"704": "Error con la contraseña de la llave Privada",
"708": "No se pudo conectar al SAT",
"711": "Error con el certificado al cancelar",
"712": "El número de noCertificado es diferente al del número de"
"certificado del atributo certificado",
}
class ParamsPac(models.Model):
_inherit = 'params.pac'
_inherit = "params.pac"
@api.model
def _get_pac_type(self):
types = super(ParamsPac, self)._get_pac_type()
types.extend([
('pac_finkok', _('PAC Finkok')),
])
types.extend([("pac_finkok", _("PAC Finkok"))])
return types
pac_type = fields.Selection(
_get_pac_type,
'Process to perform', type='char', size=64, required=True,
help='Type of process to configure in this pac',
"Process to perform",
type="char",
size=64,
required=True,
help="Type of process to configure in this pac",
)
def get_driver_fc_sign(self):
Show 20 lines Show all unchanged lines Show 20 lines
factura_mx_type__fc = super(ParamsPac, self).get_driver_fc_sign()
if factura_mx_type__fc is None:
factura_mx_type__fc = {}
factura_mx_type__fc.update({'pac_finkok': self.sign_cfdi_finkok})
factura_mx_type__fc.update({"pac_finkok": self.sign_cfdi_finkok})
return factura_mx_type__fc
def get_driver_fc_cancel(self):
Show 20 lines Show all unchanged lines Show 20 lines
factura_mx_type__fc = super(ParamsPac, self).get_driver_fc_cancel()
if factura_mx_type__fc is None:
factura_mx_type__fc = {}
factura_mx_type__fc.update({'pac_finkok': self.cancel_cfdi_finkok})
factura_mx_type__fc.update({"pac_finkok": self.cancel_cfdi_finkok})
return factura_mx_type__fc
def sign_cfdi_finkok(self, fdata):
Show 20 lines Show all unchanged lines Show 20 lines
# I thought it was necessary to create at first
# a Soap request but only passing XML in base64 it works
contenido = server.service.stamp(fdata, user, password)
# I do not know what are the error generated for Finkok
# that's why I catch all, in the other hand I do not expect
# to be here never
except: # noqa
# Send only a raise
# in order to allow higher levels catch errors
raise
# Exceptions could be MethodNotFound, PortNotFound
# ServiceNotFound, TypeNotFound, BuildError, SoapHeadersNotPermitted
# and WebFault, we only catch the generic one.
except (
MethodNotFound,
PortNotFound,
ServiceNotFound,
TypeNotFound,
BuildError,
SoapHeadersNotPermitted,
WebFault,
) as e:
user avatar user avatar user avatar +1
raise UserError(e)
# If there is Incidencias attribute it is because
# process failed
if contenido.Incidencias:
Show 20 lines Show all unchanged lines Show 20 lines
code_error = incidencia.CodigoError
if code_error in pac_exceptions.validation_error_list:
raise pac_exceptions.ValidationError(
code=code_error, message=menssage_error,
code=code_error, message=menssage_error
)
else:
raise Exception(' '.join([code_error, menssage_error]))
raise Exception(" ".join([code_error, menssage_error]))
# I think, if there is not any Incidencia it is because
# all worked well, then I have my XML properly signed in SAT
else:
return {'cfdi_xml': contenido.xml}
return {"cfdi_xml": contenido.xml}
def cancel_cfdi_finkok(self, uuid):
"""Funcion uses to cancel a CFDI in SAT server
Show 20 lines Show all unchanged lines Show 20 lines
certificate = self.company_id.certificate_id
if not certificate:
raise UserError(
_('There is no VALID certificate for company: {company}!\n'
'It is possible that the CERTIFICATE is already expired.\n'
'Go to Configuration >> Companies >> Companies to verify '
'it.').format(company=self.company_id.name),
_(
"There is no VALID certificate for company: {company}!\n"
"It is possible that the CERTIFICATE is already expired.\n"
"Go to Configuration >> Companies >> Companies to verify "
"it."
).format(company=self.company_id.name)
)
company_rfc = self.company_id.partner_id.vat_split
cer_file = certificate.certificate_file_pem.encode('base64')
key_file = certificate.certificate_key_file_pem.encode('base64')
cer_file = certificate.certificate_file_pem.encode("base64")
key_file = certificate.certificate_key_file_pem.encode("base64")
cfdis = [uuid]
try:
Show 20 lines Show all unchanged lines Show 20 lines
# I thought it was necessary to create at first
# a Soap request but only passing XML in base64 it works
# The next lines are needed by the python suds library
cfdis_list = server.factory.create('UUIDS')
cfdis_list = server.factory.create("UUIDS")
cfdis_list.uuids.string = cfdis
result = server.service.cancel(
cfdis_list, user, password, company_rfc, cer_file, key_file,
cfdis_list, user, password, company_rfc, cer_file, key_file
)
# Exceptions could be MethodNotFound, PortNotFound
# ServiceNotFound, TypeNotFound, BuildError, SoapHeadersNotPermitted
# and WebFault, we only catch the generic one.
except (
MethodNotFound,
PortNotFound,
ServiceNotFound,
TypeNotFound,
BuildError,
SoapHeadersNotPermitted,
WebFault,
) as e:
raise UserError(e)
# Get results, result is always in a list
# but this list has only one value since we only send one
# folio fiscal
folio = result.Folios.Folio[0]
result_status = str(folio.EstatusUUID)
if result_status not in ["200", "201", "202"]:
raise pac_exceptions.CancelError(
result_status, _cancel_errors.get(result_status, "")
)
# Get results, result is always in a list
# but this list has only one value since we only send one
# folio fiscal
folio = result.Folios.Folio[0]
result_status = str(folio.EstatusUUID)
if result_status not in ['200', '201', '202']:
raise pac_exceptions.CancelError(
result_status, _cancel_errors.get(result_status, ''),
)
# I do not know what are the error generated for Finkok
# that's why I catch all, in the other hand I do not expect
# to be here never
except: # noqa
# Send only a raise
# in order to allow higher levels catch errors
raise