Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fetch cardReference during completeAuthorize #6

Open
judgej opened this issue Feb 28, 2018 · 3 comments
Open

Fetch cardReference during completeAuthorize #6

judgej opened this issue Feb 28, 2018 · 3 comments

Comments

@judgej
Copy link
Member

judgej commented Feb 28, 2018

If a card reference is being created when authorizing (by setting the createCard flag) then the card reference must be fetched as a separate request for all payment types except Payment Page.

This change is needed to avoid the need to make that second, explicit API call. During CompleteRequest::send() an API call is made to fetch the card details and supplement the data already returned to the merchant site with the end user. The createCard option needs to be set when completing the transaction in order to trigger this functionality.

@judgej
Copy link
Member Author

judgej commented Feb 28, 2018

Completed for the 2.x branch.

@judgej
Copy link
Member Author

judgej commented Mar 12, 2018

This should be extended to the notify message too, so the card reference is available to the back end without the need to perform an explicit separate request to get the reference.

judgej added a commit that referenced this issue Aug 23, 2018
No documentation yet, and we may need to restrict this to just
credit and debit card payment types, but it's working to try out.
@judgej
Copy link
Member Author

judgej commented Aug 23, 2018

Making an additional request in the complete response feels messy, since a new $gateway needs to be built to support it. But it seems to be the only way to do this. The request is only run on getCardReference().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant