Skip to content
This repository has been archived by the owner on Jul 22, 2020. It is now read-only.

Evita lanzar excepción si no encuentra domicilio fiscal de receptor. #8

Merged

Conversation

apux
Copy link
Contributor

@apux apux commented Aug 26, 2015

Valida que exista el nodo //Domicilio antes de tratar de procesarlo.

@unRob
Copy link
Owner

unRob commented Aug 26, 2015

Gracias por el PR @apux, pero cómo puedes ver no pasan los tests, y por otro lado, creo que el domicilio fiscal del receptor es un campo requerido para generar un CFDI válido.

@unRob unRob added the parsing label Aug 26, 2015
Valida que exista el nodo antes de tratar de procesarlo.
@apux apux force-pushed the domicicio-fiscal-de-receptor-no-obligatorio branch from 9e645b2 to d90dbd2 Compare August 26, 2015 22:06
@apux
Copy link
Contributor Author

apux commented Aug 26, 2015

@unRob el domicilio fiscal no es un campo requerido para el cfdi, de hecho, yo tengo facturas timbradas sin un domicilio fiscal, por eso es que me di cuenta del problema con la gema.
Las pruebas efectivamente, no pasan, pero no es por mi PR, fallan desde master. Le estoy dando una vista a ver si encuentro el motivo.

@unRob
Copy link
Owner

unRob commented Aug 27, 2015

¡Tienes razón! el domicilio fiscal es completamente opcional, voy a revisar el build y hago tu merge después. Gracias por el PR.

unRob added a commit that referenced this pull request Aug 27, 2015
@unRob unRob merged commit bacbbbd into unRob:master Aug 27, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants