Skip to content

GitLab

Something went wrong on our end
Merged
Opened by Cuauhtémoc Díaz Minor@cuauhtemoc.diazContributor

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

All threads resolved
Viewing commit 7ea5eb6b
1 file
+ 14
- 14
Show latest version
l10n_mx_params_pac/pacs/finfok.py
+ 14
- 14
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
# 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:
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
result = server.service.cancel(
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
Show 20 lines Show all unchanged lines Show 20 lines
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