X-Git-Url: http://git.freeside.biz/gitweb/?a=blobdiff_plain;f=LinkPoint.pm;h=fb686829065dbc9d4c4dd272d2618ae1a95a596f;hb=HEAD;hp=1e5b1abb4c8a46e18e07330ca5b45ac096bd9551;hpb=d723cf7eb99572f3b4ac17a649bace0c9177b52e;p=Business-OnlinePayment-LinkPoint.git

diff --git a/LinkPoint.pm b/LinkPoint.pm
index 1e5b1ab..fb68682 100644
--- a/LinkPoint.pm
+++ b/LinkPoint.pm
@@ -1,20 +1,14 @@
 package Business::OnlinePayment::LinkPoint;
 
-# $Id: LinkPoint.pm,v 1.26 2007-04-07 00:00:31 jeff Exp $
-
 use strict;
 use vars qw($VERSION @ISA $DEBUG @EXPORT @EXPORT_OK);
 use Carp qw(croak);
-use AutoLoader;
 use Business::OnlinePayment;
 
-require Exporter;
-
-@ISA = qw(Exporter AutoLoader Business::OnlinePayment);
-@EXPORT = qw();
-@EXPORT_OK = qw();
-$VERSION = '0.07';
-$DEBUG = 1;
+@ISA = qw(Business::OnlinePayment);
+$VERSION = '0.11';
+$VERSION = eval $VERSION; # modperlstyle: convert the string into a number
+$DEBUG = 0;
 
 use lpperl; #3;  #lpperl.pm from LinkPoint
 $LPPERL::VERSION =~ /^(\d+\.\d+)/
@@ -130,7 +124,7 @@ sub submit {
       $month = '0'. $month if $month =~ /^\d$/;
     }
 
-    $content{'address'} =~ /^(\S+)\s/;
+    $content{'address'} =~ /^(\d+)\s/;
     my $addrnum = $1;
 
     my $result = $content{'result'};
@@ -141,10 +135,15 @@ sub submit {
       $result ||= 'LIVE';
     }
 
-    #docs disagree with lpperl.pm here
-    $content{'voidcheck'} = 1       
-      if ($self->transaction_type() =~ /^e?check$/i
-          &&  $content{'action'} =~ /^VOID$/);
+    #strip phone numbers of non-digits for ACH/echeck
+    #as per undocumented suggestion from LinkPoint
+    if ( $self->transaction_type =~ /^e?check$/i ) {
+      foreach my $field (qw( phone fax )) {
+        $content{$field} =~ s/\D//g;
+      }
+    }
+    # stuff it back into %content
+    $self->content(%content);
 
     $self->revmap_fields(
       host         => \( $self->server ),
@@ -152,13 +151,21 @@ sub submit {
       #storename    => \( $self->storename ),
       configfile   => \( $self->storename ),
       keyfile      => \( $self->keyfile ),
-      addrnum      => \$addrnum,
+
+      chargetotal  => 'amount',
       result       => \$result,
+      addrnum      => \$addrnum,
+      oid          => 'order_number',
+      ip           => 'customer_ip',
+      userid       => 'customer_id',
+      ponumber     => 'invoice_number',
+      comments     => 'description',
+      #reference_number => 'reference_number',
+
       cardnumber   => 'card_number',
       cardexpmonth => \$month,
       cardexpyear  => \$year,
-      chargetotal  => 'amount',
-      oid          => 'order_number',
+
       bankname     => 'bank_name',
       bankstate    => 'bank_state',
       routing      => 'routing_code',
@@ -185,7 +192,7 @@ sub submit {
       host port configfile keyfile
       result
       chargetotal cardnumber cardexpmonth cardexpyear
-      name email phone addrnum city state zip country
+      name company email phone fax addrnum city state zip country
       oid
       dl dlstate routing account accounttype bankname bankstate name void
 
@@ -193,6 +200,11 @@ sub submit {
 
     $post_data{'ordertype'} = $content{action};
 
+    #docs disagree with lpperl.pm here
+    $post_data{'voidcheck'} = 1       
+      if $self->transaction_type() =~ /^e?check$/i
+          && $post_data{'ordertype'} =~ /^VOID$/;
+
     if ( $content{'cvv2'} ) { 
       $post_data{cvmindicator} = 'provided';
       $post_data{cvmvalue} = $content{'cvv2'};
@@ -200,20 +212,31 @@ sub submit {
 
     if ( $DEBUG ) {
       warn "$_ => $post_data{$_}\n" foreach keys %post_data;
+      $post_data{debug} = 'true';
     }
 
-    my %response;
+    $post_data{'cargs'} = '-k -m 300 -s -S'; #if $self->test_transaction;
+
+    # avoid some uninitialized warnings in lpperl.pm
+    foreach (qw(webspace debug debugging)) { $post_data{$_} ||= '' }
+
+    #my %response;
     #{
     #  local($^W)=0;
     #  %response = $lperl->$action(\%post_data);
     #}
-    %response = $lperl->curl_process(\%post_data);
+    my %response = $lperl->curl_process(\%post_data);
 
     if ( $DEBUG ) {
       warn "$_ => $response{$_}\n" for keys %response;
     }
 
-    if ( $response{'r_approved'} eq 'APPROVED' ) {
+    if ( $response{'r_approved'} eq 'APPROVED'
+         or ( $self->transaction_type() =~ /^e?check$/i
+              && $response{'r_approved'} eq 'SUBMITTED'
+            )
+       )
+    {
       $self->is_success(1);
       $self->result_code($response{'r_code'});
       $self->authorization($response{'r_ref'});
@@ -222,7 +245,11 @@ sub submit {
     } else {
       $self->is_success(0);
       $self->result_code('');
-      $self->error_message($response{'r_error'});
+      if ( $response{'r_error'} =~ /\S/ ) {
+        $self->error_message($response{'r_error'});
+      } else {
+        $self->error_message($response{'r_approved'}); # no r_error for checks
+      }
     }
 
 }
@@ -277,8 +304,10 @@ For detailed information see L<Business::OnlinePayment>.
 
 =head1 COMPATIBILITY
 
-This module implements an interface to the LinkPoint Perl Wrapper
+This module implements an interface to the LinkPoint Perl Wrapper "lpperl",
+which you need to download and install separately.
 http://www.linkpoint.com/product_solutions/internet/lperl/lperl_main.html
+http://www.linkpoint.com/viewcart/down_index.htm
 
 Versions 0.4 and on of this module support the LinkPoint Perl Wrapper version
 3.5.
@@ -289,6 +318,10 @@ Versions 0.4 and on of this module support the LinkPoint Perl Wrapper version
 
 Ivan Kohler <ivan-linkpoint@420.am>
 
+Contributions from Mark D. Anderson <mda@discerning.com>
+
+Echeck work by Jeff Finucane <jeff@cmh.net>
+
 Based on Busienss::OnlinePayment::AuthorizeNet written by Jason Kohles.
 
 =head1 SEE ALSO