search

wmixvideo / nfe

Nota Fiscal Eletrônica em Java.
Apache License 2.0
644 stars 376 forks source link

Criado validação para nao permitir números negativos #917

Closed giovaneboeing closed 11 months ago

sonarcloud[bot] commented 1 year ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

fincatto commented 1 year ago

Oi @giovaneboeing. Estou em duvida se ali é o melhor lugar para colocar esse validador, pois é um parse de bigdecimal, que pode ter valores negativos. O ideal não seria a aplicação tratar isso no retorno do objeto já parseado?

giovaneboeing commented 1 year ago

Olá @fincatto Também é uma possibilidade, mas como não vi nenhuma regra onde é permitido valor negativo, decidi por diretamente na lib. Se tiver algum caso para me passar onde é possível utilizar valores negativos, pode me passar e cancelar a PR. Eu faço as alterações em minha aplicação dai.

sonarcloud[bot] commented 11 months ago

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

fincatto commented 11 months ago

Acredito que esta validação não deva estar aqui, por engessar o metodo.