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 10 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