Create AccesDataLaravel

stavarengo / php-sigep

PHP client library for the Brazilian Post Office Web Service. Integração com Web Service do Correios. Consulta preços e prazos, imprime etiquetas e PLP, etc.

https://phpsigep.stavarengo.me

MIT License

363 stars 249 forks source link

Create AccesDataLaravel #336

Closed barreirinha closed 5 years ago

barreirinha commented 5 years ago

Um AccesData para quem usa Laravel, pemitindo a configuração dos dados através de variáveis de ambiente.

andersonls commented 5 years ago

@barreirinha Não vi nenhuma amarração desta classe com o framework, me parece que é uma classe muito específica da tua aplicação. Nesse caso não sei se vale a pena dar merge, o que acha @stavarengo ?

barreirinha commented 5 years ago

Não, simplifica bastante a chamada. Masta usar:

$AccesData = new AccesDataLaravel(); (para ambiente de Testes)
$AccesData = new AccesDataLaravel(true); (para ambiente de Produção)

Com isso, cada usuário pode colocar seus dados nas variáveis de ambiente do Laravel, que são amplamente usadas.

Comecei a fazer o ServerProvider, mas ainda farei ajustes antes de subir.

stavarengo commented 5 years ago

Oi @barreirinha . Obrigado pelo PR. O PHP Sigep é focado em comunicação com a API dos Correios. O AccessData é uma classe necessária para se comunicar com o serviço do Correios, porem, a forma como vc popula o AccessData é responsabilidade de cada projeto. Seguindo este raciocínio essa alteração, apesar de facilitar o trabalho de alguns usuários, está fora do escopo do projeto.