API-3.0-PHP icon indicating copy to clipboard operation
API-3.0-PHP copied to clipboard

Adiciona suporte para desativar, reativar, alterar o dia de cobrança e valor de pagamentos recorrentes

Open murillobit opened this issue 6 years ago • 5 comments

Acrescentei uma uma classe base para tratar de requisições de atualização de recorrências e 4 novos métodos para implementar as seguintes operações:

  • Desativar recorrência
  • Reativar recorrência
  • Alterar data de cobrança da recorrência
  • Alterar o valor da recorrência

Precisei remover a restrição de tipo JsonSerializable do parâmetro content do método sendRequest, pois na atualização de recorrências, algumas requisições (ex: alterar data e valor) enviam apenas um inteiro no body da requisição (https://developercielo.github.io/manual/cielo-ecommerce#modificando-o-valor-da-recorr%C3%AAncia). Essa restrição causa erro de execução no código. Dei uma pesquisada para saber qual seria a melhor saída e, para evitar de acrescentar uma classe para implementar essa interface para inteiro no pacote, achei melhor remover a restrição.

murillobit avatar Mar 02 '18 15:03 murillobit

@muvasco Este PR já foi incorporado ao master?

marcelo2605 avatar Mar 03 '20 14:03 marcelo2605

@marcelo2605 ainda não, está parado aguardando eles darem merge...

murillobit avatar Mar 03 '20 14:03 murillobit

Acrescentei uma uma classe base para tratar de requisições de atualização de recorrências e 4 novos métodos para implementar as seguintes operações:

  • Desativar recorrência
  • Reativar recorrência
  • Alterar data de cobrança da recorrência
  • Alterar o valor da recorrência

Precisei remover a restrição de tipo JsonSerializable do parâmetro content do método sendRequest, pois na atualização de recorrências, algumas requisições (ex: alterar data e valor) enviam apenas um inteiro no body da requisição (https://developercielo.github.io/manual/cielo-ecommerce#modificando-o-valor-da-recorr%C3%AAncia). Essa restrição causa erro de execução no código. Dei uma pesquisada para saber qual seria a melhor saída e, para evitar de acrescentar uma classe para implementar essa interface para inteiro no pacote, achei melhor remover a restrição.

Fui tentar testar o método de atualizar preço e realmente deu erro aqui. Mudei a classe AbstractRequest e funcionou. Obrigado!!

ArthurYdalgo avatar Jul 31 '20 16:07 ArthurYdalgo

Eai @netojoaobatista, vai rolar esse merge? O que ainda é necessário para que esse mr seja aprovado?

wdarking avatar Dec 10 '20 21:12 wdarking

Eai @netojoaobatista, vai rolar esse merge? O que ainda é necessário para que esse mr seja aprovado?

fala, @wdarking. Sei q a pergunta n foi pra mim, mas caso te ajude:

Clonei o repo e adicionei esse (e outros) métodos. link pro repo

criei tb um pacote pro composer com esse meu repositório, caso esteja precisando desses métodos link pro pacote no packagist

ArthurYdalgo avatar Dec 10 '20 21:12 ArthurYdalgo