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

Alterado PHP Binding Classes para arrumar alguns erros de parse e para tratar melhor os erros da API #13

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

bfgasparin
Copy link

@bfgasparin bfgasparin commented Feb 23, 2017

Atenção: Eu ainda não terminei de testar as alterações feitas nessa pull request. Não fazer o merge ainda. Criei a pull request para compartilhar o trabalho com vocês.

Motivo da Pull Request

Ao tentar usar a biblioteca PHP, tive diversos problemas como erros de parse de json , modelos de objetos fora do padrão do contrato da API da Nfe.io, .... Além disso, muitos erros na comunicação com a API (HTTP 409, HTTP 400, ...) eram camuflados e ignorados na biblioteca, dando um falso positivo de que tudo ocorreu bem para quem usava a mesma.

Abaixo segue a lista das alterações feitas na biblioteca.

Peço para que analisem, e se fizer sentido, façam o merge da full request para que futuros usuários não tenham os mesmo problemas que eu tive.

Allterações

  • Correção de leitura do body da response em APIRequest
  • Melhoria nos tratamentos de erros da API em APIRequest
  • Correção na construção das binding classes em APIResource
  • Criação de novos exceptions
  • Correção do parser da resposta da API em Utilities

Outras Melhorias

A biblioteca está um pouco defasada em termos de arquitetura de código. Não segue PSR e a arquitetura poderia estar melhor desenhada.

Esta pull request não tem como objetivo atacar estes pontos de melhoria.

@bfgasparin bfgasparin changed the title Alterado PHP Find Classes para arrumar alguns erros de parse e tratar melhor os erros da API Alterado PHP Binding Classes para arrumar alguns erros de parse e para tratar melhor os erros da API Feb 23, 2017
@renatonascalves
Copy link
Contributor

@bfgasparin Quando terminar, me marca que gostaria de fazer o code review. :)

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