ecomplus / app-wirecard

E-Com Plus app to integrate Wirecard BR
MIT License
1 stars 1 forks source link

Admin settings desatualizado e campo não funcional #29

Closed leomp12 closed 3 years ago

leomp12 commented 3 years ago

O admin settings publicado aqui não corresponde ao que é instalado atualmente (publicado no Market), nele temos um campo list_payments (label Ordem de Pagamento) que parece não estar funcionando, inclusive não encontrei no código lugar onde ele é tratado.

Adicionalmente:

talissonf commented 3 years ago

Isso foi um bug no market que ta cagando o admin_settings quando é alterado via UI. https://github.com/ecomplus/market/issues/37

talissonf commented 3 years ago

https://market.e-com.plus/v2/applications/1238

leomp12 commented 3 years ago

Hmm, mas na verdade a intenção não é remover o campo do admin settings, mas sim tratá-lo devidamente na reposta para o módulo list_payments. O lojista tentou usá-lo e não fez efeito, removê-lo do admin settings não resolve o problema.

No mínimo responda primeiro com cartão de crédito, em geral é a primeira opção de pagamento e é o que o lojista está tentando configurar.

talissonf commented 3 years ago

aquele campo foi sobrescrito, ele nao serve de nada la

leomp12 commented 3 years ago

Tô ligado que ele não serve :smile: Mas o negócio é que ainda precisamos de um campo para ordenar as formas de pagamento (o nome da propriedade faz...), vi que tem um sort mas este está funcional? Também não achei nada no código tratando ele.

talissonf commented 3 years ago

https://github.com/ecomplus/app-wirecard/commit/82a4912ac5be327e22c68fa2fc59d3bc1a28a596

leomp12 commented 3 years ago

@talissonf você testou? Seu enum no admin settings indica 'Cartão de Crédito' e no código você seta 'Cartão de crédito', no sort você não normaliza o case da string então acho que não vai funcionar.

talissonf commented 3 years ago

testei sim

leomp12 commented 3 years ago

Hmm até funciona mas porque na verdade o cartão (último no array por padrão) fica em primeiro mesmo se config.sort for 'abc' ou qualquer outra coisa diferente de 'Boleto Bancário' e 'Débito Online'.

'Cartão de Crédito'.indexOf('Cartão de crédito') = -1

Selecionando a opção Cartão de Crédito sua função no sort retorna 1 em todos os testes porque config.sort.indexOf(a.payment_method.name) dá -1 pras 3 formas de pagamento, nesse caso o último item do array vira primeiro...