It would be great to have a drupal 7 version of this module.

Comments

dgtlmoon’s picture

StatusFileSize
new28.11 KB

First attempt attached here

NOTE: I have dropped support for the reoccuring/registered accounts payments, this is a single-hit payment method only, once the maintainers come back to life or I get access to the project we'll lock this is as 7.x and build it back up

also incorporates patch from #1542886: does not properly pass cvv information to payment gateway

dgtlmoon’s picture

StatusFileSize
new28.1 KB

Oops, always the way, fixed a glitch in $order->currency handling

rooby’s picture

Nice, I was going to do this, so I will definitely try it out soon.

I will be needing recurring support though so I will look at adding that once I have tried it out.

dgtlmoon’s picture

Great! I've removed the handling of different txn_type's so you'll need to trace it through from the top and compare it against the D6 version.

Perhaps move the recurring related functions into their own include file to keep it clean

univate’s picture

Great stuff D7 port of this module is long overdue...

+++ b/uc_securepayau.moduleundefined
@@ -81,17 +75,12 @@ function uc_securepayau_settings_form() {
-  $form['settings']['uc_securepayau_currency'] = array(
-    '#type' => 'select',
-    '#title' => 'Currency',
-    '#default_value' => variable_get('uc_securepayau_currency', variable_get('uc_currency_code', 'AUD')),
-    '#options' => array('AUD' => 'AUD', 
-                        'USD' => 'USD'),
-  );

Why disable the currency support? Its a useful feature to have since securepay actually supports this really easily if you have a NAB multi-currency account. Although the project I used this on only cared about USD support, so I didn't expand the list here to include all currencies that securepay/NAB can support.

+++ b/uc_securepayau.moduleundefined
@@ -213,300 +208,6 @@ function _uc_securepayau_charge($order, $amount, $data) {
 /**
-* Implementation of hook_recurring_info().

The core API For uc_recurring has not changed for D7 (I know as I am the maintainer for uc_recurring) - so this should all be able to be added and hopefully just work. Not against the idea of splitting into a separate file for maintenance reasons.

deciphered’s picture

Status: Active » Needs review
StatusFileSize
new26.05 KB

Patch didn't apply, so I re-rolled it and took the comments from #5 into account.

I was investigating using uc_recurring, but it turned out that it's unlikely to meet my clients needs, so while there is at least one minor fix for that functionality, I can't confirm that it is all working.

dgtlmoon’s picture

Status: Needs review » Closed (fixed)

Your patch was almost identical to something I had, comitted and formed new 7.x-dev branch