search

artesaos / moip

Pagamentos Moip
https://moip.com.br/
MIT License
38 stars 19 forks source link

Atualizar a dependência moip/moip-sdk-php para versão mais recente. #36

Open lucassena opened 5 years ago

lucassena commented 5 years ago

Estou usando o package artesaos/moip (v1.0.3) no Laravel 5.4, mas vi que a dependência da moip/moip-sdk-php está usando 2.2.*, só que estou tendo alguns problemas com Exceptions e vi que foi corrigido nas últimas versões.

Gostaria de saber se vocês podem atualizar a dependência ou então terei que fazer um fork para eu poder atualizar essa dependência e resolver meu problema aqui.

juniorjc commented 5 years ago

@lucassena qual problema que você está tendo com Exceptions ?

SOSTheBlack commented 5 years ago

@lucassena Realmente preciso fazer esse updade, pretendo fazer nos próximos dias

Se está precisando isso de modo emergencial, aconselho a você adicionar a dependência do moip no composer.json do seu projeto no require que automaticamente está package passará a utilizar a versão mais atual, mesmo ele requerendo uma versão anterior

lucassena commented 5 years ago

@lucassena qual problema que você está tendo com Exceptions ?

Fala @juniorjc, eles corrigiram esse problema na versão 3.1.0.

(Se quiser ver o commit https://github.com/wirecardBrasil/moip-sdk-php/commit/3862cc69)

lucassena commented 5 years ago

@lucassena Realmente preciso fazer esse updade, pretendo fazer nos próximos dias

Se está precisando isso de modo emergencial, aconselho a você adicionar a dependência do moip no composer.json do seu projeto no require que automaticamente está package passará a utilizar a versão mais atual, mesmo ele requerendo uma versão anterior

Cara, eu tentei fazer isso, tentei um composer require moip/moip-sdk-php, mas ele deu um erro dizendo que o artesaos/moip requer a versão 2.2.*. Veja a imagem:

image

Fora isso tem um problema com o novo resource Holder do SDK.

De qualquer forma, eu fiz aqui o update em um fork, adicionei o método para instanciar Resource\Holder e parece estar funcionando bem, estou testando os pagamentos aqui. Se quiser dou um PR para você dar uma olhada @SOSTheBlack.

EDIT: Então @SOSTheBlack, eu terminei os testes aqui com cartão de crédito e boleto para verificar se está tudo certo. Parece estar tudo certo, vou enviar o PR e assim vocês podem decidir o que fazer a respeito.

Obrigado!

SOSTheBlack commented 5 years ago

@lucassena PR aceito https://github.com/artesaos/moip/pull/37

utilize a versão develop para ter acesso ao ultimo commit da repo. Assim que pelo os seus testes estiver tudo OK eu gero um novo release.