[ic] BUG in &charge= when passing options (IC 4.8.2)

[Javier Martin]

Mike Heins said:

> of time. The change (to Util.pm) is in CVS, both stable and devel.

Now that we are at it, I have made some changes to the AuthorizeNet.pm
module (4.8.2). I don't know whether you were aware of them. Sorry if I'm
recurring. The diff is original versus mine:


376c376
< 					x_Type			=> $actual->{cyber_mode},
---
> 					x_Type			=> $transtype,

This one is because all the mapping is performed on the variable
$opt->{transaction}. If we used $actual->{cyber_mode} as x_Type, no mapping
would take place and specifying "sale" would actually send "sale" instead of
"AUTH_CAPTURE".



386c386
<                     x_Phone        => $actaul->{phone_day},
---
>                     x_Phone        => $actual->{phone_day},

Just a typo.



446,447c446,447
<
< #::logDebug(qq{authorizenet response_reason_text=$response_reason_text
response_code: $response_code});
---
>
> #::logDebug(qq/authorizenet
response_reason_text=$result{x_response_reason_text} response_code:
$result{x_response_code}/);

Those variables ($response_reason_text and $response_code) are never defined
and would not produce any useful debug output.


Javier