laravel-pagseguro icon indicating copy to clipboard operation
laravel-pagseguro copied to clipboard

Validação não acontece em todos os itens do pedido.

Open renatosistemasvc opened this issue 7 years ago • 1 comments

Me parece que existe um erro de lógica na validação que acontece na linha 291 do arquivo PagSeguro.php.

A validação não acontece em todos os itens do pedido. Se tiver 3 itens por, exemplo, ele valida o primeiro e o segundo item e não valida o terceiro.

Isso acontece, pois a comparação na linha 305 do mesmo arquivo é:

for ($cont = 1; $cont < $this->itemsCount; $cont++) {

mas deveria ser

for ($cont = 1; $cont <= $this->itemsCount; $cont++) {

Eu usei a sua biblioteca como base para criar a minha própria, pois precisava usar o modelo de aplicação (que é quando o cliente me dá permissão para receber pagamento por ele). Essa forma de integração é melhor pois em um clique o usuário integra a conta dele com o meio de pagamento. Caso queira que eu desenvolva essa nova funcionalidade aqui neste library me diga como poderia contribuir com o projeto. grande abraço!

renatosistemasvc avatar Aug 22 '17 03:08 renatosistemasvc

Já vou corrigir esta parte da validação dos itens, obrigado por avisar. Edit: 1.4.2

Eu estou estudando uma maneira de dividir a biblioteca em classes menores para facilitar a reutilização do código (e a criação de testes), pelo que eu vi a parte das aplicações envia as mesmas requisições praticamente, mas utilizando o app key, id e authorization code, então acho que seria interessante dividir melhor as classes pra conseguirmos ter esta feature sem duplicarmos o código.

Vou trabalhar nisso ao longo da semana 😄, se tiver como passar um exemplo de como ficou a sua biblioteca ja seria uma grande ajuda =)

fernandobandeira avatar Aug 22 '17 16:08 fernandobandeira