Skip to content
This repository has been archived by the owner on Mar 16, 2024. It is now read-only.

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

Open
lucassena opened this issue Apr 25, 2019 · 5 comments
Open

Comments

@lucassena
Copy link
Contributor

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
Copy link
Contributor

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

@SOSTheBlack
Copy link
Member

@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
Copy link
Contributor Author

@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 wirecardBrasil/moip-sdk-php@3862cc69)

@lucassena
Copy link
Contributor Author

lucassena commented Apr 27, 2019

@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
Copy link
Member

@lucassena PR aceito
#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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants