maxipago / sdk-php

maxiPago! Smart Payments - PHP library for our payment gateway
http://www.maxipago.com
Other
14 stars 20 forks source link

Esta SDK está muito antiga (sem namespace, sem recursos modernos do PHP) #21

Open vitorccs opened 7 months ago

vitorccs commented 7 months ago

Pessoal

Puxa vida... que falta de investimento por parte da MaxiPago aqui.

Essa biblioteca precisa: 1) Ser versionada - para que os sistemas que irão usar, possam determinar a versão compatível e evitar surpresas desagradáveis 2) Precisa usar PSR e suas recomendações - para que pelo menos ela seja compatível com namespace e importação via Composer, ninguém mais importa biblioteca dessa maneira 3) Utilizar Composer para gerenciar dependências - puxa vocês colocaram a bilbioteca KLogger diretamete no código-fonte, não há como administrar as atualizações dela (incluindo as de segurança) 4) Usar PHP moderno (ex: PHP 8.x) para se beneficiar de recursos modernos, como a tipagem de propriedades que o código atual faz de modo totalmente manual. 5) Criar testes de unidade em alguma biblioteca moderna como PHPUnit ou Pest 6) Orientação o objeto - pouco agrega fazer uma SDK que apenas encaminha o payload puro em array para a API, vocês precisam trabalhar com o conceito de builder para facilitar a criação do payload ou algo similar

Fontes: https://semver.org/lang/pt-BR/ https://www.php-fig.org/psr/psr-0/ https://www.php-fig.org/psr/psr-4/

vitorccs commented 4 months ago

A quem interessar...

Segue SDK que estou desenvolvendo com as boas práticas levantadas acima.

https://github.com/vitorccs/maxipago-php-sdk

Obs: No momento, ele consegue fazer pagamentos PIX e Boleto Bancário, mas em futuro breve irei implementar Cartão de Crédito.