I wanted to check with you about what plans you might have had for integrating with uc_echecknet (http://drupal.org/project/uc_echecknet)? I cloned the uc_echecknet module and combined that with code from your authnet_uc module to get Ubercart working with saved CIM bank accounts (http://drupal.org/node/1766012) and with non-saved bank accounts (using AIM). In order to get this to work, I needed to patch the authnet_ui module to only return account types of 'bankAccount'.
I'm thinking that the uc_echecknet patch I have needs to be posted in that module's issue queue, but I would love to get your thoughts first.
Thanks!
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | authnet_uc_echecknet_1896910_1.patch.txt | 1.52 KB | pfrilling |
Comments
Comment #1
pfrillingThe attached authnet_ui patch needs to be applied after the patch in #7 here: http://drupal.org/node/1766012#comment-6939400
This patch will allow us to filter the type of account that is returned. This is useful for authnet_uc to only display saved credit cards. This is also useful for integrating with uc_echecknet as we didn't want credit card accounts showing when a user wants to pay with echeck.
Comment #2
m.stentaI doubt that we would integrate with uc_echecknet, unless absolutely necessary. A better approach would be for our core Authorize.net module to add support for processing echeck.net payments, and then add add code to the authnet_uc module to make that possible in Ubercart. I'm not very familiar with the way echeck.net payments work, however, so it would require some research. Any light you can shed on the matter would be greatly appreciated!
Comment #3
m.stentaOur comments crossed... I posted before seeing your comment in #1.
I like this patch... makes sense. I still haven't had time to review the other patch you posted (http://drupal.org/node/1766012#comment-6939400), but I hope to soon. It seems like they will go hand-in-hand.
Comment #4
pfrillingProcessing an echeck payment using AIM is almost the same for both credit cards and echecks. The major difference is in the array we pass to the authnet_aim_transaction() function.
The _authnet_uc_aim_transaction() function has the $sale object with the following options.
To get an ECHECK to process we remove the options above and use the following options instead:
As for CIM payments, both credit cards and bank accounts work the same.
My clients needs right now are for ECHECK and CIM to be integrated into their website. So, I went ahead and cloned the uc_echecknet module and made it work with the authnet module using both AIM and CIM based upon this patch (http://drupal.org/node/1766012#comment-6939400). I guess my question is: does it make more sense to roll the cloned module I have completed in with the authnet api OR keep the uc_echecknet module separate but dependent upon the authnet api module?
Let me know your thoughts.
Comment #5
m.stentaI applied this patch (FYI it still applies cleanly against the new patches I just added: http://drupal.org/node/1766012#comment-6990170 and http://drupal.org/node/1900244#comment-6990172).
Questions/comments:
Thanks again @pfrilling for all your work on this!
Comment #6
m.stentaJust a heads up: the Ubercart integration module (authnet_uc) has been split out into a separate contrib: http://drupal.org/project/uc_authnet.
That shouldn't affect the patch in #1, but it will affect the other things you're doing probably.
Comment #7
m.stentaMoving this to 7.x-1.x queue because Drupal 6 is EOL tomorrow: https://www.drupal.org/drupal-6-eol