Split PE - Set the order's payment method ID to the processing payment method #2951
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #2949
Changes proposed in this Pull Request:
This PR makes sure we set the order's payment method ID to the UPE payment method used to process the payment. Prior to our changes in
add/deferred-intent
this code use to set tostripe
all the time (code link). In #2883, I changed it to set the orders payment ID to the retrievable payment method ID (ie SEPA for bancontact, iDEAL, Sofort). That PR was about subscription support so I suspect I was under the impression thats how it should work for orders, however that's not true.Testing instructions
add/deferred-intent
note the order's payment method is set to SEPA.I've tested subscription purchases and this still correctly sets the subscription's payment method SEPA and the parent order to Bancontact.
Order Payment Method is iDEAL
Subscription Payment Method is SEPA debit
changelog.txt
andreadme.txt
(or does not apply)Post merge