Followup to #2946909: When a payment gateway error occurs, the order does not become unocked.
There we fixed exceptions thrown from PaymentProcess, but not the PaymentGatewayForm handling, which will need a different fix due to not having access to the $order.

CommentFileSizeAuthor
#3 2958309-3.patch11.06 KBbojanz
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bojanz created an issue. See original summary.

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Assigned: Unassigned » bojanz

Thanks to the changes in 2.12 (https://www.drupal.org/node/3015309) this issue can now be fixed much easier, since the offsite form is now built with the main form in PaymentProcess, not afterwards when processing the main form.

bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Status: Active » Needs review
FileSize
2958309-3.patch11.06 KB

  • bojanz committed 714e120 on 8.x-2.x 
    Issue #2958309 by bojanz: The order is not unlocked when an exception is...
bojanz’s picture

bojanz CreditAttribution: bojanz at Centarro commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.