Novos parâmetros adicionados ao Construtor, sem valor default, quebrou a implementação ao atualizar o SDK

SafeToPay / PHP

SDK do Safe2Pay para a linguagem PHP.

14 stars 16 forks source link

Novos parâmetros adicionados ao Construtor, sem valor default, quebrou a implementação ao atualizar o SDK #15

Open frankdors opened 3 years ago

frankdors commented 3 years ago

Mais de 100 pagamentos não foram processados devido a atualização!!

Favor colocar valor default nas próximas implementações. Favor adicionar tags de versionamento para facilitar analise das mudanças controle da versão do sdk no composer

Too few arguments to function Safe2Pay\Models\Payment\CreditCard::__construct(), 5 passed in /var/www/html/sistema/action/gateway/safe2pay.php on line 92 and exactly 9 expected</b> (Safe2Pay #0)

https://github.com/SafeToPay/PHP/compare/31022c...master

rafwell commented 3 years ago

@frankdors sugiro que crie um fork do projeto e utilize o seu repositório, assim você terá um controle melhor, pois o repositório oficial não está nem com SEMVER implementado.

Esse problema de adicionarem novos campos no construtor sem valor padrão já ocorreu pelo menos 2 vezes comigo, inclusive da última adicionaram variáveis no meio do construtor, podendo gerar graves problemas de integração.

gabrielvgarcia commented 3 years ago

@rafwell @frankdors Versão atualizada com valores default no cartão de crédito para campos opcionais, seguiremos esse padrão nas próximas atualizações.