nfe / client-php

Official NFe.io API Client for PHP
https://nfe.io
MIT License
40 stars 18 forks source link

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

Open bfgasparin opened 7 years ago

bfgasparin commented 7 years ago

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

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.

renatonascalves commented 6 years ago

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