CFDI icon indicating copy to clipboard operation
CFDI copied to clipboard

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

Open apux opened this issue 9 years ago • 1 comments

@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?

apux avatar Sep 17 '15 22:09 apux

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 avatar Dec 29 '16 18:12 unRob