search

Sylius / PayPalPlugin

Official integration with PayPal Commerce Platform
37 stars 56 forks source link

[Payment] fix cancel payment #273

Closed oallain closed 9 months ago

oallain commented 1 year ago
Q A
Branch? 1.5
Bug fix? yes
New feature? no
Related tickets replace #249

If the order already has a payment in the "new" status, then the Sylius orderPaymentProcessor must not be processed, otherwise a new payment in the "cart" status will be created, which will cause errors (2 payments in order ...).

Before : image

After : image

oallain commented 1 year ago

Hello @Zales0123 @SirDomin and others ;)

I fixed the bug that made two new payments (one in new and one in cart) when cancelling a payment. Because of this, the order was broken.

Regards

oallain commented 1 year ago

Hello,

This bug is still present, so I've rebased my branch to make it compatible with the new version.

Can you take a look? Ping @GSadee

GSadee commented 9 months ago

Thank you, Olivier! :1st_place_medal: