Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hook onGetPaymentMeans destroys payment selection in Customer Account #494

Open
thomasdcodes opened this issue May 4, 2023 · 1 comment

Comments

@thomasdcodes
Copy link

Moin!
There is a Hook (onGetPaymentMeans) in Shopware\Plugins\MoptPaymentPayone\Subscribers\Payment.php which should combine the credit cards. But it also destroys the payment selection in the customer account.
In your God-Class PayonePaymentHelper.php in the method groupCreditcards() is the Id of the Payment-Method in Line 1477 (!!!) set. But that is not working for the /account/payment Route.
Furthermore all other Payments will be deleted in Line 1465-1467:
if ($firstHit != $key) { unset($paymentMeans[$key]); }
in the loop over all payments.
Please fix that behavior as soon as possible.

@thomasdcodes
Copy link
Author

PUSH

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant