search

allgood / APInter-PHP

Biblioteca PHP para acesso à API do Banco Inter
GNU Lesser General Public License v3.0
104 stars 52 forks source link

Return type of jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize() com o Laravel #30

Closed dadeke closed 2 years ago

dadeke commented 2 years ago

Olá Claudemir

Eu novamente! 😁 Como vai?

Estou utilizando sua lib com o Laravel e ao rodar os testes estão aparecendo essas mensagens abaixo.

Note que separadamente todos os testes do seu projeto está rodando 100% ✅ Isso acontece apenas quando está junto com o Laravel. Tudo indica que precisa desses ajustes para ficar compatível.

Pergunta: Posso tentar corrigir isso e lhe enviar um pull request?

PHP version

8.1.6

Laravel version

Framework 9.15.0

PHP Deprecated:  Return type of ctodobom\APInterPHP\Cobranca\Pagador::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /usr/share/nginx/html/sistema/modules/InterBr/vendor/ctodobom/api-inter/src/Cobranca/Pagador.php on line 263
PHP Deprecated:  Return type of ctodobom\APInterPHP\Cobranca\Boleto::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /usr/share/nginx/html/sistema/modules/InterBr/vendor/ctodobom/api-inter/src/Cobranca/Boleto.php on line 355
PHP Deprecated:  Return type of ctodobom\APInterPHP\Cobranca\Mensagem::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /usr/share/nginx/html/sistema/modules/InterBr/vendor/ctodobom/api-inter/src/Cobranca/Mensagem.php on line 92
PHP Deprecated:  Return type of ctodobom\APInterPHP\Cobranca\Desconto::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /usr/share/nginx/html/sistema/modules/InterBr/vendor/ctodobom/api-inter/src/Cobranca/Desconto.php on line 84
PHP Deprecated:  Return type of ctodobom\APInterPHP\Cobranca\Multa::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /usr/share/nginx/html/sistema/modules/InterBr/vendor/ctodobom/api-inter/src/Cobranca/Multa.php on line 79
PHP Deprecated:  Return type of ctodobom\APInterPHP\Cobranca\Mora::jsonSerialize() should either be compatible with JsonSerializable::jsonSerialize(): mixed, or the #[\ReturnTypeWillChange] attribute should be used to temporarily suppress the notice in /usr/share/nginx/html/sistema/modules/InterBr/vendor/ctodobom/api-inter/src/Cobranca/Mora.php on line 80
PHP Deprecated:  Exception::__construct(): Passing null to parameter #2 ($code) of type int is deprecated in /usr/share/nginx/html/sistema/modules/InterBr/vendor/ctodobom/api-inter/src/BancoInterException.php on line 15
allgood commented 2 years ago

@dadeke ,

consegue testar com a última alteração na master? (ainda não portei para a branch api7)

dadeke commented 2 years ago

Executei os testes e restou apenas a mensagem:

PHP Deprecated:  Exception::__construct(): Passing null to parameter #2 ($code) of type int is deprecated in /usr/share/nginx/html/sistema/modules/InterBr/vendor/ctodobom/api-inter/src/BancoInterException.php on line 15
allgood commented 2 years ago

Acho que corrigi esse também, editei direto no github, favor testar.

Devo portar todos esses ajustes para a API 2 nos próximos dias, e provavelmente já vou soltar ela como estável.

dadeke commented 2 years ago

Acho que corrigi esse também, editei direto no github, favor testar.

Devo portar todos esses ajustes para a API 2 nos próximos dias, e provavelmente já vou soltar ela como estável.

Agora junto ao Laravel está 100% ✅

Obrigado 👍