The deferred transaction status was not being sent over to PayPoint.

Patch attached....

thanks
Leon

CommentFileSizeAuthor
#3 uc_paypoint_deferred_transactions2.patch656 bytesmarc.groth
#2 uc_paypoint_deferred_transactions.patch650 bytesmarc.groth
uc_paypoint.module.deferred.patch906 bytesleon kessler

Comments

marc.groth’s picture

marc.groth commented
Status: Active » Needs review

Thanks for this Leon,

I haven't had a chance to test this myself yet but setting the status to "needs review" so that any others can test/verify. I will test it as soon as I have a spare moment.

Cheers,

Marc

marc.groth’s picture

marc.groth commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new uc_paypoint_deferred_transactions.patch650 bytes

Thanks Leon,

I've managed to test the patch and all appears to be working fine. For some reason I had to apply the patch manually (did you create it against the latest snapshot?) so I have re-patched it, which I have attached. I'll double check with psynaptic, but since this is one line of code (and hardly any formatting) it should get committed soon.

Thanks again for all your work.

Cheers,

Marc

marc.groth’s picture

marc.groth commented
StatusFileSize
new uc_paypoint_deferred_transactions2.patch656 bytes

For some reason the formatting got messed up with the one above, re-attaching.

marc.groth’s picture

marc.groth commented
Status: Reviewed & tested by the community » Fixed

This has been committed and is now in the latest dev release.

Thanks Leon.

Marc

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.