Needs work
Project:
Paypal Payment for Invoice
Version:
6.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Feature request
Assigned:
Reporter:
Created:
19 Nov 2009 at 09:39 UTC
Updated:
22 Nov 2009 at 18:09 UTC
Jump to comment: Most recent file
Comments
Comment #1
cweagansOh! I'll look at migrating to that at some point.
Right now, I just kind of needed to get this out the door and it does the basics.
In the future, for sure though. Thanks for the pointer!
Comment #2
cweagansChanged my mind. Simple Payments really is simple enough for everyday use. I hit a couple of barriers with it, but I think it's a great module :)
Patch attached. Could you take a look at the approach taken and verify that it was the correct way to go (or not)?
Thanks!
Comment #3
cweagansWhoops. Wrong patch.
Comment #4
jbrown commenteddrupal_get_destination() doesn't return a path, so can't be passed in the first parameter of url.
just do url('node/' . $node->nid, array('absolute' => TRUE))
simple_payments is designed to attach an nid to a payment - just set vars['nid'] to $node->nid and then flag based on $payment['nid'].
You need to verify that the correct amount was paid in the payment hook. simple_payments does not cryptographically sign the payment vars.