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

[Order] Allow set credit card to order. #206

Closed
makasim opened this issue Oct 5, 2014 · 5 comments · Fixed by #327
Closed

[Order] Allow set credit card to order. #206

makasim opened this issue Oct 5, 2014 · 5 comments · Fixed by #327

Comments

@makasim
Copy link
Member

makasim commented Oct 5, 2014

addition to #204

@alexandernst
Copy link
Contributor

What about this one @makasim ? Any PR for it?

@makasim
Copy link
Member Author

makasim commented Mar 12, 2015

nobody worked on that

@alexandernst
Copy link
Contributor

But... isn't possible to pay with a VISA/MasterCard now? (using PayPal)
I'm not really sure if I understand this issue.

@makasim
Copy link
Member Author

makasim commented Mar 12, 2015

The issue is pretty simple: The order interface https://github.com/Payum/Payum/blob/master/src/Payum/Core/Model/OrderInterface.php must have a setter getter method for a credit card: https://github.com/Payum/Payum/blob/master/src/Payum/Core/Model/CreditCardInterface.php

And later in a place like https://github.com/Payum/Payum/blob/master/src/Payum/AuthorizeNet/Aim/Action/FillOrderDetailsAction.php#L22 this would allow to set credit card details. It is needed to avoid a credit card form on capture action when you have it already

@makasim
Copy link
Member Author

makasim commented Mar 12, 2015

But... isn't possible to pay with a VISA/MasterCard now? (using PayPal)

yes it is possible if you use paypal specific format, but it is partly possible with the unified interface (order interface).

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

Successfully merging a pull request may close this issue.

2 participants