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

Argument 2 passed to Moip\Resource\Payment::setCreditCardHash() must be an instance of Moip\Resource\Holder, instance of Moip\Resource\Customer given #332

Open
andrewalkermo opened this issue Oct 25, 2021 · 3 comments · May be fixed by #333

Comments

@andrewalkermo
Copy link

andrewalkermo commented Oct 25, 2021

Não consigo passar um Customer como holder no método setCreditCardHash. No phpDoc diz que o tipo é \Moip\Resource\Customer, mas no método está restrito ao tipo \Moip\Resource\Holder.

Tem sempre que criar um Holder com os mesmos dados do Customer.

É necessário remover a restrição nos métodos setCreditCardHash e setCreditCardHolder.

@andrewalkermo andrewalkermo changed the title Argument 2 passed to Moip\Resource\Payment::setCreditCardHash() must be an instance of Moip\Resource\Holder, instance of Moip\\Resource\Customer given Argument 2 passed to Moip\Resource\Payment::setCreditCardHash() must be an instance of Moip\Resource\Holder, instance of Moip\Resource\Customer given Oct 25, 2021
@mx-advaud
Copy link

mx-advaud commented Oct 26, 2021

Dá uma olhada nesse PR https://github.com/wirecardBrasil/moip-sdk-php/pull/330/files
Inclusiva já existe ao menos mais uma issue sobre isso.

@andrewalkermo
Copy link
Author

@mx-advaud Não é a mesma issue. Esse PR está se referindo ao problema de versão do rmccue/requests. O problema aqui é a restrição do tipo Holder em alguns métodos, que acredito que também deveriam aceitar o tipo Customer.

@mx-advaud
Copy link

Desculpe a demora. Realmente confundi , deveria ter respondido em outra issue.

@andrewalkermo andrewalkermo linked a pull request Jan 27, 2023 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging a pull request may close this issue.

2 participants