Skip Menu |

This queue is for tickets about the Business-OnlinePayment-AuthorizeNet CPAN distribution.

Report information
The Basics
Id: 22075
Status: resolved
Priority: 0/
Queue: Business-OnlinePayment-AuthorizeNet

People
Owner: ivan-pause [...] 420.am
Requestors: mschwern [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Unimportant
Broken in:
  • 3.15
  • 3.00
  • 3.01
  • 3.10
  • 3.11
  • 3.12
  • 3.13
  • 3.14
Fixed in:
  • 3.16
  • 3.17



Subject: Minor fixes to AuthorizeNet.pm
The attached patch makes small fixes to AuthorizeNet.pm * Eliminate inheriting from AutoLoader. We're not using it and it just screws up the error messages. * account_type mentioned twice in the required fields for checks. * Quiet an uninit value warning when customer_org is not set.
Subject: au.patch
=== AuthorizeNet.pm ================================================================== --- AuthorizeNet.pm (revision 19623) +++ AuthorizeNet.pm (local) @@ -9,7 +9,7 @@ require Exporter; -@ISA = qw(Exporter AutoLoader Business::OnlinePayment); +@ISA = qw(Exporter Business::OnlinePayment); @EXPORT = qw(); @EXPORT_OK = qw(); $VERSION = '3.15'; @@ -151,10 +151,12 @@ push @required_fields, qw( amount routing_code account_number account_type bank_name - account_name account_type + account_name ); - if ($self->{_content}->{customer_org} ne '') { + if (defined $self->{_content}->{customer_org} and + length $self->{_content}->{customer_org} + ) { push @required_fields, qw( customer_org customer_ssn ); } else { push @required_fields, qw(license_num license_state license_dob);
thanks! checked into CVS, will be 3.16 On Fri Oct 13 05:27:00 2006, MSCHWERN wrote: Show quoted text
> The attached patch makes small fixes to AuthorizeNet.pm > > * Eliminate inheriting from AutoLoader. We're not using it and it
just Show quoted text
> screws up the error messages. > > * account_type mentioned twice in the required fields for checks. > > * Quiet an uninit value warning when customer_org is not set.