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

¿No se deben generar atributos para el hash si el valor es nulo? #12

Open
apux opened this issue Sep 17, 2015 · 1 comment
Open

¿No se deben generar atributos para el hash si el valor es nulo? #12

apux opened this issue Sep 17, 2015 · 1 comment
Labels

Comments

@apux
Copy link
Contributor

apux commented Sep 17, 2015

@unRob para generar el xml se valida que no se generen nodos si el valor no existe (es nulo)

https://github.com/unRob/CFDI/blob/master/test/comprobante_spec.rb#L55-L63

¿Sería recomendable el mismo comportamiento para generar el hash?

Por ejemplo:

# en lugar de:
{lugar_expedicion: nil, subtotal: 123.0}
# generar solamente los que sí tienen valores
{subtotal: 123.0}

¿Te interesa un PR en ese sentido?

@unRob
Copy link
Owner

unRob commented Dec 29, 2016

Chale, qué pena @apux por ignorar este issue tanto tiempo! No estoy seguro de cual deba de ser el comportamiento adecuado, para ser honesto, pero creo que ir de atributos con valores nulos a un hash sin los mismos no debe de ser tanto rollo, tipo hash.delete_if { |k, v| v.nil? }.

@unRob unRob added the question label Dec 29, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants