FLUX-SE / PayumStripe

Payum Stripe gateways (with SCA support)
MIT License
28 stars 15 forks source link

Add extra check for cancelled payment_intent #32

Closed JoppeDC closed 1 year ago

JoppeDC commented 1 year ago

Fixes https://github.com/FLUX-SE/SyliusPayumStripePlugin/issues/46

Prometee commented 1 year ago

Thank you again @JoppeDC, are you able to add a PHPUnit test for this specific case ?

For the record : this case is happening when a CheckoutSession object is synced to it's related PaymentIntent. While being into the action : StatusSessionAction.php the Sync will change the model object to a PaymentIntent object, so we have to stop here and re-execute the request which will execute the StatusPaymentIntentAction instead.

JoppeDC commented 1 year ago

@Prometee No problem! Glad i could help debug the issue.

Unfortunatly i feel like i lack some knowledge on the specific business logic here to be able to quickly add a test.