2 replies [Last post]
StephenGWills's picture
Offline
Uber DonorBug FinderEarly adopter... addicted to alphas.Getting busy with the Ubercode.Not Kulvik
Joined: 08/07/2007
Juice: 414
Was this information Helpful?

The first thing I do when I take an update is change line 134 of uc_authorize.net.module from

<?php

'x_type' => 'AUTH_CAPTURE',

to

'x_type' => 'AUTH_ONLY',
?>

I wonder if this should be an option of the authorize.net module in the admin setup instead of a hack. Does anybody else, using authorize.net, use it just to screen cards as opposed to capturing funds?

Steve

StephenGWills's picture
Offline
Uber DonorBug FinderEarly adopter... addicted to alphas.Getting busy with the Ubercode.Not Kulvik
Joined: 08/07/2007
Juice: 414
Re: Authorize.net option to AUTH_ONLY instead of AUTH_CAPTURE

also, I don't know if this is a settable parameter in the merchant section of authorize.net or a bug in the module but I need to change line 201 of uc_authorize.net.module:

<?php
FROM
:
$response = split('\,', $authorize);
TO:
$response = split('\|', $authorize);
?>

If this is a settable feature at authorize.net, it should probably also be settable in the admin settings for the module?

Steve

Ryan's picture
Offline
Joined: 08/07/2007
Juice: 15438
Re: Re: Authorize.net option to AUTH_ONLY instead of AUTH_CAPTUR

Honestly, I don't understand this:

<?php
  $response
= split('\,', $authorize);
 
$response_code = explode(',', $response[0]);
 
$response_text = explode(',', $response[3]);
 
$approval_code = explode(',', $response[4]);
 
$x_response_code = $response_code[0];
 
$x_response_text = $response_text[0];
 
$x_approval_code = $approval_code[0];
?>

I do see in my tests that values are coming back with , delimiter and not |, but I also see in their demo that it comes back with |. So I don't know where this is set. However, I also don't get why there's a separate explode in there after the split. Sticking out tongue Gonna toy with it. In the meantime I added the possibility to set it to AUTH_ONLY, and I will make the delimiter configurable.

EDIT: meh, I switched it to just use one explode()... works fine.