This issue is just to track the changes required to ubercart to get uc_recurring working with the most current version of ubercart. Once the api changes have settled down it will be transfered to the ubercart project to be committed.

The changes are just to the payment methods specifically and you will need this patch if you are using either of these modules to process recurring payments:
* uc_credit
* uc_payment_pack

At the moment there are separate issues for changes to the following payment gateways:
* uc_paypal
* uc_authorizenet (this is dependent on uc_credit changes here)

CommentFileSizeAuthor
ubercart_uc_recurring.patch6.57 KBunivate

Comments

univate’s picture

univate commented
Status: Active » Needs review
ac’s picture

ac
Primary language English
Location Perth
commented

subscribing and reviewing

reubenavery’s picture

reubenavery commented

Hi,
There were still numerous problems I was finding and ended up putting quite a bit of time into a pretty thorough attempt at getting this uc_recurring 2.x working with uc 2.x and authorize.net ARB.

I put two mutually dependent patches for uc_recurring and various uc core modules up over yonder: http://drupal.org/node/454952#comment-1730126

I thought it would be best to keep them together due to a change I made in hook_order

matt2000’s picture

matt2000 commented

sub

brentratliff’s picture

brentratliff commented

Giving it a shot with uc_credit and the Linkpoint API module.

brentratliff’s picture

brentratliff commented

Abandoning Linkpoint for Authorize.net, Linkpoint will not work behind load balancers.

socialnicheguru’s picture

socialnicheguru commented

subscribing

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented

@univate,
I think this whole patch can and should live in uc_recurring. Since this module is now in contrib, ubercart doesn't need to implement it's hooks.
A few module_exists('uc_credit') or seperating the code to include files can do the job. For example:


/**
 * Callback to process a recurring renewal
 */
function uc_recurring_test_gateway_renew($order, $fee) {

  if (module_exists('test_gateway)) { // <- uc_recurring implements it's own hooks.
    if ($key = uc_credit_encryption_key()) {
      ...
    }
    return $result['success'];
    }
  }
}
univate’s picture

univate commented

Thats an interesting idea...

The one issue I have is that I don't want uc_recurring to be tied to any specific payment gateways, I see this module as a an API for payment gateways to implement recurring payments/billing in ubercart.

At the moment ubercart includes a number of payment methods/payments (which I think probably shouldn't be there, but thats a different issue), a number of these have ways of implementing recurring functionality: paypal, authorizenet, google_checkout, 2checkout - are you suggesting we should include all the recurring payment interfaces for these as well in uc_recurring?

amitaibu’s picture

amitaibu
Primary language Hebrew
Location Israel
commented

> The one issue I have is that I don't want uc_recurring to be tied to any specific payment gateways, I see this module as a an API for payment gateways to implement recurring payments/billing in ubercart.

Think of it as Rules module is doing. It provides an API for other modules, but also implements actions/ conditions for Drupal core's module.
uc_recurring could be the same - providing an API, and in a subfolder, it will implement Ubercart core's payment modules.

jday’s picture

jday commented

I would love to test this out, I have

UC 6.x-2.0-rc3
Recurring Fees 6.x-2.x-dev
and an Authorize.net ARB account

do I need the patch from the head of this thread on the dev version for recurring fees to work?

bearstar’s picture

bearstar commented

I have everything working OK using authorize.net and CIM but I wonder when I would expect to see items listed at /admin/store/orders/recurring

Would authorize.net Test mode transactions show up there?

murokoma’s picture

murokoma commented

I am a bit confused here - the patch is for the test gateway, right? But doesn't it say in the beginning that this is for the uc_credit module?
Sorry about this, I'm neither into patches nor programming :-)

Thanks for your help!

murokoma’s picture

murokoma commented

I'm sorry, I got it when performing the test, asked for the other two files and patched them correctly.
However, after uploading them nothing seems to happen; is there any "idiot-proof" explanation on WHAT to do ? :-)

univate’s picture

univate commented

The patch here is going to become redundant when this patch is committed:
#534296: Make uc_recurring completely independant from ubercart, uc_recurring will handle all ubercart core payment methods via inc files

univate’s picture

univate commented
Status: Needs review » Closed (won't fix)

This is no longer required, uc_recurring now includes the uc_credit and test_gateway code so you can test out this module without any need to patch ubercart.