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

A classe toolsNfe deveria ser totalmente removida #925

Merged
merged 1 commit into from
Dec 31, 2016

Conversation

billbarsch
Copy link
Contributor

@billbarsch billbarsch commented Dec 30, 2016

A classe toolsNfe deveria ser totalmente removida do arquivo porém continuava aparecendo nele.
Gerou erros para gerar a danfe em dois pontos:
1-o vTotTrib que no meu xml estava nulo e dava erro.
2-na linha da presença da toolsNfe

Adotei colocar uma logo "padrão" já direto no php. (logo tirada do google imagens)
Só sugestão, fiquem a vontade para analisar e adotar as alterações.

A classe toolsNfe deveria ser totalmente removida do arquivo porém continuava aparecendo nele.
Gerou erros para gerar a danfe em dois pontos:
1-o vTotTrib que no meu xml estava nulo e dava erro.
2-na linha da presença da toolsNfe

Adotei colocar uma logo "padrão" já direto no php. (logo tirada do google imagens)
Só sugestão, fiquem a vontade para analisar e adorar as alterações.
@robmachado robmachado merged commit 8317970 into nfephp-org:master Dec 31, 2016
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