search

Payum / PayumBundle

Payum offers everything you need to work with payments. From simplest use cases to very advanced ones.
https://payum.gitbook.io/payum
MIT License
569 stars 142 forks source link

Support for Digital Goods Variables #7

Closed rat4m3n closed 11 years ago

rat4m3n commented 11 years ago

Would this be possible to support Digital Goods variables please?

    // Set REQCONFIRMSHIPPING to 0
    // Set NOSHIPPING to 1.
    // Set category L_PAYMENTREQUEST_0_ITEMCATEGORYn = Digital
makasim commented 11 years ago

add demo to sandbox app too. https://github.com/Payum/PayumBundleSandbox/commit/fc6e7eb04077a609b01393f48d77d7545bdc637f

rat4m3n commented 11 years ago

Demo works fine with file storage but fails with Doctrine .... error is showing it can't set Return url

vendor/payum/paypal-express-checkout-nvp/src/Payum/Paypal/ExpressCheckout/Nvp/Api.php at line 222

And I can't track whats wrong with Doctrine implementation....

rat4m3n commented 11 years ago

Ah after latest changes I think we are missing some mapping definitions here:

PaypalExpressCheckoutNvp / src / Payum / Paypal / ExpressCheckout / Nvp / Bridge / Doctrine / Resources / mapping / PaymentInstruction.orm.xml

For return / cancel urls and whatever is required

PS. Maybe allow auto mapping... ?

makasim commented 11 years ago

Ah after latest changes I think we are missing some mapping definitions here:

I've just added an example to sandbox (paypal express checkout + doctrine), It works fine.

For return / cancel urls and whatever is required

it is not required field. I add mapping only for really needed fields like token, errors and so on. If you want store some other info you can easy add mapping for them in your model.

makasim commented 11 years ago

yeap I saw the problem too. It could happen when you do a redirect to capture controller. right?

so there two possible solution:

rat4m3n commented 11 years ago

Nice one! Works like a charm @makasim ! Yes, I was using a redirect...

Thanks @makasim