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

omnipaybridge example #7

Closed
ghost opened this issue Feb 28, 2014 · 4 comments
Closed

omnipaybridge example #7

ghost opened this issue Feb 28, 2014 · 4 comments

Comments

@ghost
Copy link

ghost commented Feb 28, 2014

Does anyone have a working example for using OmniPaybridge with PayumModule?

It seems like they should work together, and I can get OmniPaybridge working outside of PayumModule and I can I can PayumModule working ok with paypal express (and have put a few sites live using this code), I just can't work out how to make them work together.

Ideally with the Sagepay Server gateway, but at this point i'd settle for any of the Omnipay gateways!

@makasim
Copy link
Member

makasim commented Feb 28, 2014

You have to create a Payment instance using PaymentFactory from omnipay bridge. It will ask you for omnipay gateway. Later in prepare action configure payment details as omnipay tells you to do it and do the payment. Two places differs from paypal:

  • configuration
  • prepare action

rest has to be same

@ghost
Copy link
Author

ghost commented Feb 28, 2014

Thanks. I guess I must be doing something wrong in the prepare action then, as it's definitely creating the right payment instance (in my case SagePay_Server) and then i'm getting a strange "Omnipay unexpected exception" exception when I pass the parameters in and then make the redirect to the capture token URL.

@makasim
Copy link
Member

makasim commented Mar 1, 2014

@tmaul I think it is thrown here https://github.com/Payum/OmnipayBridge/blob/master/Action/CaptureAction.php#L49. Check previous exception for more details

@makasim
Copy link
Member

makasim commented Mar 12, 2014

exception wrapping was removed in master

@makasim makasim closed this as completed Mar 12, 2014
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