[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
Changes proposed in this Pull Request:
In the last update, we stopped storing donation amounts in $_SESSION. This appeared to work ok (correct amount shown in PayPal Express checkout UI and in PMPro orders), but in reality, the donation amount was not actually being charged when you look at the completed transactions in PayPal. This is due to how our confirmation step works after a PayPal express session is completed.
This PR fixes that issue by pulling the donation amount that the user chose out of the PMPro order when performing the PPE confirmation step to ensure that we are "confirming" the correct amount (level cost + donation).
Resolves XXX.
How to test the changes in this Pull Request:
Set up Donations Add On
Set up PPE
Check out for level with donation via PPE
Look at completed transaction in PPE. See that before PR, amount without donation shows. With PR, correct amount shows.
Other information:
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you successfully run tests with your changes locally?
Changelog entry
Enter a summary of all changes on this Pull Request. This will appear in the changelog if accepted.
All Submissions:
Changes proposed in this Pull Request:
In the last update, we stopped storing donation amounts in
$_SESSION. This appeared to work ok (correct amount shown in PayPal Express checkout UI and in PMPro orders), but in reality, the donation amount was not actually being charged when you look at the completed transactions in PayPal. This is due to how our confirmation step works after a PayPal express session is completed.This PR fixes that issue by pulling the donation amount that the user chose out of the PMPro order when performing the PPE confirmation step to ensure that we are "confirming" the correct amount (level cost + donation).
Resolves XXX.
How to test the changes in this Pull Request:
Other information:
Changelog entry