Skip to content

Conversation

@bezidejni
Copy link
Contributor

Hi guys,

This is a fix for some of the methods in the PayPal Express facade that are tying to get the transaction ID by the wrong key name and subsequently fail later when sending a request to PayPal with a missing transaction ID.

codeinthehole added a commit that referenced this pull request Sep 2, 2014
Fix for Express facade refund/capture/void methods
@codeinthehole codeinthehole merged commit f28e019 into django-oscar:master Sep 2, 2014
@codeinthehole
Copy link
Contributor

Good catch - thank you.

Not sure why I was using TRANSACTIONID instead of PAYMENTINFO_0_TRANSACTIONID. PayPal's API did tend to duplicate values so perhaps I was using the key that has since been dropped.

I'll cut a new point release with this fix as it's fairly critical.

@codeinthehole
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants