Closed danilobreda closed 5 months ago
@silvairsoares tudo certo? esse é o fork do Zeus, Vamos analisar seu pullrequest em breve! Obrigado pela contribuição!
Caso achem a solução interessante, me compromento a fazer um novo PR, marcando as demais tags da NF-e/NFC-e que são do tipo TString. É basicamente um trabalho braçal, mas muito extenso. São centenas de tags desse tipo espalhadas por todo o schema do XML.
Aqui vai meu pensamento sobre a proposta:
Não sei se é algo que o ZeusFiscal deveria se preocupar. Se pegamos a responsabilidade de que formatamos a string da melhor maneira, isso pode ser um problema muito grande para nós como responsabilidade e mantenedores. Pensando alto: E se tiver um bug e formatarmos o CFOP incorretamente e o cliente emitir tudo errado. Naturalmente o programador resolve isso no sistema dele dando um trim().
A remoção de acentuação é uma opção hoje que faz +- isso, porem é feito diretamente no XML, não propriedade por propriedade. Achei complexo a implementação :/ e principalmente para dar manutenção
Minha opinião segue a do @danilobreda.
Entretanto acho um Add-On legal... que tal um projeto de extensão @silvairsoares ?
Minha opinião segue a do @danilobreda.
Entretanto acho um Add-On legal... que tal um projeto de extensão @silvairsoares ?
Concordo com a opinião de vocês. Realmente, se abrir precedente pra este tipo de tratamento, propriedade por propriedade, os usuários destas soluções já teriam esta expectativa e seria algo muito pesado de se manter, além de uma baita responsabilidade. Aqui, eu implementei esta funcionalidade no nosso SDK que comunica com a API de emissão de notas, e não diretamente nesta API. Vou avaliar esta possibilidade de criar um Add.On, porém acho difícil ser viável, uma vez que a minha proposta exige que as propriedades das classes usadas para gerar os payloads com os dados dos DFE-s, sejam decorados, e estas classes, já está implementadas dentro do próprio Hercules/ZeusFiscal. De qualquer forma, agradeço pelo tempo dispensado na alálise da minha proposta.
Vou fechar, se aparecer alguma ideia diferente sobre esse assunto não deixe de compartilhar com a gente. Muito obrigado pela contribuição @silvairsoares
Pullrequest de @silvairsoares no Zeus puxado para cá.