I intend to use this module on a future version of my website. However, I needed an extra feature which is presetting allowed payment types such that users can optionally select within a subset of payment types before being redirected to the Paybox site. I used the same mechanism implemented in the core 'uc_credit' module to set the choice as a session variable.

I also fixed a small bug which prevented the 'Back' link to be a link (a missing >)

Here is the patch for review and eventually integration.

CommentFileSizeAuthor
#3 uc_paybox.patch8.56 KBpvasener
uc_paybox.patch8.54 KBpvasener

Comments

mogtofu33’s picture

Assigned: Unassigned » mogtofu33
Status: Needs review » Fixed

Hi,

Thanks a lot for the patch, it has been integrate on beta2.

Regards,
Mog.

Status: Fixed » Closed (fixed)

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

pvasener’s picture

Status: Closed (fixed) » Reviewed & tested by the community
StatusFileSize
new8.56 KB

Thanks for the 1.0 version. However, I've noticed that my patch, which was included in beta2 was not in 1.0. Besides, it's not applying for the stable version due to a comment addition. So, here's an updated version of it for v1.0, hoping it will be included in future releases.

pvasener’s picture

Version: 6.x-1.0-beta1 » 6.x-1.0

Changing Version.

mogtofu33’s picture

Status: Reviewed & tested by the community » Fixed

Hi,

Sorry, it was an error, patch has been lost in outer space (?). Patch apply on uc_paybox 6.x-1.1
Thanks a lot.

Regards.
Mog

Status: Fixed » Closed (fixed)

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