Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Garantindo que instancia de serviço na CTe esteja sempre instanciada e seja utilizada corretamente #26

Merged

Conversation

JoaoDiasDev
Copy link
Contributor

Em alguns casos, alguns processos como cancelamento da CTe estava gerando exception devido a isto, verifiquei que tinham mais lugares necessitando de tratamento.

Procurei garantir que instancia do serviço sempre seja usada corretamente, como é nulavel o parâmetro ConfiguracaoServico configuracaoServico em alguns lugares estava chegando nulo, dependendo do fluxo da CTe utilizado, fiz um tratamento padrão em todos lugares que poderia acontecer, apesar de em alguns métodos de validação que são usados em vários lugares já tratar de instanciar novamente o serviço, muitos lugares não instanciava, gerando exception em alguns métodos. Acredito que tenha tratado todos lugares onde é nulavel o parâmetro configuracaoServico.

…te, como é nulavel o parametro ConfiguracaoServico configuracaoServico em alguns lugares estava chegando nulo, dependendo do fluxo da CTe utilizado, fiz um tratamento padrão em todos lugares que poderia acontecer, apesar de em alguns metodos de validação que são usados em varios lugares já tratar de instanciar novamente o serviço, muitos lugares não instanciava, gerando exception em alguns metodos. Acredito que tenha tratado todos lugares onde é nulavel o parametro.
@JoaoDiasDev JoaoDiasDev changed the title Garantindo que instancia de serviço na CTe esteja sempre instanciado e seja utilizada corretamente Garantindo que instancia de serviço na CTe esteja sempre instanciada e seja utilizada corretamente Jul 16, 2024
@marcosgerene
Copy link
Collaborator

@JoaoDiasDev

Muito obrigado pela contribuição!

@marcosgerene marcosgerene merged commit 92c5480 into Hercules-NET:master Jul 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants