ifthenpay / mbway-magento

1 stars 0 forks source link

Fix alteração de estado #1

Open ruipns opened 4 years ago

ruipns commented 4 years ago

Faz com que a alteração do estado de pagamento apenas seja efectuada se o pagamento for efectuado por MbWay; Outros métodos de pagamento como o PayPal geram encomenda em estado "processing" o que faria com que estado fosse alterado para "Pending Payment" e causaria conflito com o order flow das encomendas

LeoNun commented 4 years ago

Bom dia, atualizei ontem este modulo do MBWay, de facto agora já modifica corretamente o estado, no entanto ao criar o envio a encomenda é colocada como estado "Fechado" e não "Finalizado". Confirma esta situação?

Cumprimentos, Leonel Nunes

ruipns commented 4 years ago

Boa tarde, Em relação à finalização da encomenda não houve alterações na proposta que realizei, com a alteração de ontem não testei mas antes da alteração a encomenda é criada em pending, quando efetuado o pagamento passa a processing e quando é criado o envio passa a complete. A proposta que realizei apenas salvaguarda outros métodos de pagamento que geram encomendas corretamente em processing devido ao pagamento já estar realizado, sem a verificação que coloquei passaria a pending incorretamente.

O único problema que surgiu foi novamente na alteração de estado de encomenda inicial, a encomenda é gerada em processing e é colocada em pending posteriormente o que faz conflito com outros módulos que correr quando uma encomenda é guardada e verificam o estado. Notei isto com o sistema de faturação, o módulo corre quando a encomenda é guardada e visto que é criada a processing era gerada uma fatura para uma encomenda ainda não paga, a encomenda passava a pending e quando era efetuado o pagamento passa novamente a processing o que fazia com que fosse gerada novamente uma fatura para a mesma encomenda. Se facilitar posso criar uma issue sobre este tema, no meu caso arranjei um workaround da situação mas pode ser conveniente a outros clientes.

Cumprimentos, Rui Silva

LeoNun commented 4 years ago

Boa tarde Sr. Rui, agradeço desde já o seu email.

De facto tenho sempre o mesmo problema, quando crio o envio da encomenda após esta ser paga e faturada esta passa para o estado de “Fechado” e não o estado correto que é “Finalizado”. Envio um print screen em anexo, de facto este é o unico método de pagamento que me está a terminar as encomendas desta forma.

Com os melhores cumprimentos,


Leonel Nunes T: 914 656 508


From: ruipns Sent: Thursday, June 4, 2020 7:18 PM To: ifthenpay/mbway-magento Cc: LeoNun ; Comment Subject: Re: [ifthenpay/mbway-magento] Fix alteração de estado (#1)

Boa tarde, Em relação à finalização da encomenda não houve alterações, a encomenda é criada em pending, quando efetuado o pagamento passa a processing e quando é criado o envio passa a complete. A proposta que realizei apenas salvaguarda outros métodos de pagamento que geram encomendas corretamente em processing devido ao pagamento já estar realizado, sem a verificação que coloquei passaria a pending incorretamente.

O único problema que surgiu foi novamente na alteração de estado de encomenda inicial, a encomenda é gerada em processing e é colocada em pending posteriormente o que faz conflito com outros módulos que correr quando uma encomenda é guardada e verificam o estado. Notei isto com o sistema de faturação, o módulo corre quando a encomenda é guardada e visto que é criada a processing era gerada uma fatura para uma encomenda ainda não paga, a encomenda passava a pending e quando era efetuado o pagamento passa novamente a processing o que fazia com que fosse gerada novamente uma fatura para a mesma encomenda. Se facilitar posso criar uma issue sobre este tema, no meu caso arranjei um workaround da situação mas pode ser conveniente a outros clientes.

Cumprimentos, Rui Silva

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

ruipns commented 4 years ago

Boa tarde,

Este pull request não é relacionado com essa questão, aconselho a abrir uma issue ou contactar o suporte.

A encomenda só deveria ficar fechada com um refund do valor ou se algum módulo a alterar para isso e que tenha conhecimento este módulo não faz em lugar algum alteração para esse estado. Posto isto suponho que seja uma incompatibilidade com a forma como está a criado o método de envio ou algo que faça com que o pagamento seja devolvido. Não consigo visualizar o screenshot e sem mais informação torna-se difícil perceber o que é, aconselho a fazer um debug de todo o workflow da encomenda e tentar perceber em que local está a ser feito a alteração para o estado de "fechada". Se precisar de ajuda adicional poderei tentar ajudar, poderá me contactar para o email ruipns@live.com.pt

Cumprimentos, Rui Silva

LeoNun commented 4 years ago

Boa noite Sr. Rui, ok, irei abrir um novo ticket com este assunto. SIm, as encomendas ficam com o estado de fechado quando existe o reembolso (nota de crédito) da encomenda, dai esta minha observação...

Agradeço-lhe uma vez mais,

Com os melhores cumprimentos,


Leonel Nunes T: 914 656 508


From: ruipns Sent: Thursday, June 4, 2020 8:11 PM To: ifthenpay/mbway-magento Cc: LeoNun ; Comment Subject: Re: [ifthenpay/mbway-magento] Fix alteração de estado (#1)

Boa tarde,

Este pull request não é relacionado com essa questão, aconselho a abrir uma issue ou contactar o suporte.

A encomenda só deveria ficar fechada com um refund do valor ou se algum módulo a alterar para isso e que tenha conhecimento este módulo não faz em lugar algum alteração para esse estado. Posto isto suponho que seja uma incompatibilidade com a forma como está a criado o método de envio ou algo que faça com que o pagamento seja devolvido. Não consigo visualizar o screenshot e sem mais informação torna-se difícil perceber o que é, aconselho a fazer um debug de todo o workflow da encomenda e tentar perceber em que local está a ser feito a alteração para o estado de "fechada". Se precisar de ajuda adicional poderei tentar ajudar, poderá me contactar para o email ruipns@live.com.pt

Cumprimentos, Rui Silva

— You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe.

luisnovaisdev commented 4 years ago

Bom dia,

Eu estou com o mesmo problema ? A alteração do código no ficheiro AfterPlaceOrderObserver.php resolve o problema ? Acontece também que são enviados dois emails de confirmação de encomenda, como é possível corrigir isso ?

LeoNun commented 4 years ago

Bom dia Sr. Luís, de facto já enviei email ao suporte da IfThenPay e irão averiguar o que se passa. Quanto ao ficheiro AfterPlaceOrderObserver.php, sim, faz todo o sentido que o problema seja solucionado por aí..

Cumprimentos, Leonel Nunes

ruipns commented 4 years ago

Bom dia,

Esta alteração apenas protege outros métodos de envio, não soluciona nada interno desde método. Em relação aos email de confirmação duplicados, confirma que esses email são enviados quando a encomenda é gravada em estado processing? Se for o caso pode tentar resolver isso com outros métodos ou tentar ver a atualização mais recente em que altera o call to invoice. Outro caso que gostaria que alguém pude-se confirmar seria se ao definir este método de pagamento apenas para Portugal se aparece para outros países, de momento está a acontecer, aparece disponível independentemente do país por isso gostaria de ter confirmação se é algo do nosso lado ou do módulo.

Cumprimentos, Rui Silva

LeoNun commented 4 years ago

Boa tarde Sr. Rui, de facto a questão dos emails não está a acontecer comigo. Quanto ao método de pagamento funcionar apenas para Portugal, confirmo-lhe que sim, mas neste momento tenho ativo para outros países.

Cumprimentos, Leonel Nunes