nfephp-org / sped-da

Classes para geração dos documentos auxiliares usados pelos padrões Sped
Other
139 stars 238 forks source link

Danfe Simplificado não mostra peso bruto em alguns casos #512

Closed matheusryndack closed 2 years ago

matheusryndack commented 2 years ago

Notei que não estava saindo o peso bruto/líquido na danfesimplificada, e averiguando encontrei esse trecho de código que pula o volume caso não tenha a tag 'esp'. (Linha 274 do arquivo DanfeSimples.php).

Esse comportamento é esperado ou é um bug? Fiquei na dúvida, porque tem um comentário especificando falando "Espécie não especificada na transportadora (por erro de preenchimento do XML)", no entanto tenho XMLs autorizados sem essa tag, e ela não é obrigatória.

Posso efetuar a correção e um Pull Request? Minha sugestão é validar a existência dos campos de peso em si, e não da tag 'esp'.

robmachado commented 2 years ago

Pode; mas seja cuidadoso para não fazer outras alterações

matheusryndack commented 2 years ago

Feito!