Dividir o gateway em dois, um para debito e outro para crédito about cielo-woocommerce HOT 17 CLOSED

A idéia seria adicionar isso debaixo da opção CIELO - um passo a mais (e.g. a pessoa seleciona cielo e depois seleciona débito ) ou separar isso e mostrar as opções (CIELO DEBITO E CIELO CRÉDITO) ?

from cielo-woocommerce.

Virar dois gateways msmo, um Cielo Crédito e outro Cielo Debito.

from cielo-woocommerce.

A gente vai implementar isso já para o 3.2 ?

from cielo-woocommerce.

Vai ser melhor para funcionar o webservice agora

from cielo-woocommerce.

Posso manter as 2 classes dos 2 gateways dentro do mesmo arquivo ? ou vc prefere separar toda a estrutura de diretórios ?

from cielo-woocommerce.

Vamos separar assim vai estruturar melhor.

from cielo-woocommerce.

Cláudio,
eu abstrai a classe WC_Cielo_Gateway e criei 2 classes herdadas que carregassem apenas os dados e características que fosse aplicados a cada contexto (crédito ou débito). Com isso eu aproveitei toda a programação e interface já programada sem precisar ter redundância.

Eu comitei o início da divisão de gateways - pelo menos o backend e a parte de load no checkout estão ok para o BuyPage; precisa testar para o webservice e verificar onde o processo de pagamento vai ser afetado com essas mudanças (carregamento das chaves de afiliacao - vide abaixo)

Algumas observações - quando a gente dá o split no gateway o único dado que ficou reduntante foi a chave de afiliação e da loja da cielo - não sei muito bem como fazer - pq "teoricamente" ele poderia ter uma chave pra débito e outra chave pra crédito ... como que a gente poderia fazer com isso ? ou deixa assim ?

from cielo-woocommerce.

Ficou bom, mas acho que podemos até fazer sem herança, porque assim diminuimos alguns ifs, maior duplicação vai ser apenas nas opções, mas isso é tranquilo.

from cielo-woocommerce.

Ou manter herança e abstrair melhor isso.

from cielo-woocommerce.

É isso não ajudou muito, da forma que você dividiu sumiu várias coisas de JS que eram importantes.
Estou precisando refazer várias coisas e dividir realmente para funcionar melhor.
Unica coisa que estou duplicado agora vai as opções.

from cielo-woocommerce.

OK - Eu tentei fazer com herança e aproveitar o máximo possível de ambos. Alguns do js estavam funcionando justamente para alterar os valores baseado se fosse debito ou credito (com a divisão eles ficaram desnecessários) - por isso que removi

Com relação ao admin - eu apenas codifiquei uns selectors mais genéricos que funcionaria para ambas as telas e criei os métodos com as opções detalhadas de cada parte (debito ou credito)

Pensei em reaproveitar ao máximo o código que já existia antes e só redefinr o que fosse estritamente necessário ;

from cielo-woocommerce.

Falo de coisas como isso aqui no default:

if ( 'discover' === card ) {
    $( 'option', installments ).not( '.cielo-at-sight' ).remove();
}

discover não recebe parcelas, só pagamento a vista.

Estou escrevendo de novo algumas coisas e melhorando isso para dividir, ter JS para cada um.

from cielo-woocommerce.

Do admin estava tudo perfeito, mas no frontend tinha coisas assim, estou agora dando uma melhorada nisso.

from cielo-woocommerce.

Agora rodando mais tranquilo.
Falta o de debit e então vai estar tudo pronto :)

from cielo-woocommerce.

Me desculpe se deu mais dor de cabeça . Vamos aprendendo e melhorando 😊 

—
Sent from Mailbox

On Sun, Mar 29, 2015 at 7:32 PM, Claudio Sanches [email protected]
wrote:

Agora rodando mais tranquilo.

Falta o de debit e então vai estar tudo pronto :)

Reply to this email directly or view it on GitHub:
#35 (comment)

from cielo-woocommerce.

Não precisa pedir desculpas, tudo de boa.
Eu juntei a classe de helper com o do gateway agora.
Separei algumas coisas e criando métodos para validar debito e crédito.

from cielo-woocommerce.

Agora foi.

from cielo-woocommerce.