X-Git-Url: http://git.freeside.biz/gitweb/?a=blobdiff_plain;f=OnlinePayment.pm;h=9b251c80c1a6972753b6541fafab2291bfef2658;hb=8936a751b7a38eedbe5044acd83a78614d82c632;hp=7c7287cffd4ba8246cb4a2a84846d97dbaab08bc;hpb=dda73c9d6b5fb43424c980323e10ceb92269c103;p=Business-OnlinePayment.git diff --git a/OnlinePayment.pm b/OnlinePayment.pm index 7c7287c..9b251c8 100644 --- a/OnlinePayment.pm +++ b/OnlinePayment.pm @@ -3,13 +3,15 @@ package Business::OnlinePayment; use strict; use vars qw($VERSION); use Carp; -use Symbol; require 5.005; -$VERSION = '3.00_04'; +$VERSION = '3.00_07'; $VERSION = eval $VERSION; # modperlstyle: convert the string into a number +# Remember subclasses we have "wrapped" submit() with _pre_submit() +my %Presubmit_Added = (); + my %fields = ( authorization => undef, error_message => undef, @@ -27,7 +29,6 @@ my %fields = ( transaction_type => undef, ); - sub new { my($class,$processor,%data) = @_; @@ -43,7 +44,7 @@ sub new { $self->build_subs(keys %fields); if($self->can("set_defaults")) { - $self->set_defaults(); + $self->set_defaults(%data); } foreach(keys %data) { @@ -54,16 +55,18 @@ sub new { $self->$key($value); } - unless ( $subclass->can('submit') eq $class->can('submit') ) { - no strict 'refs'; - no warnings 'redefine'; - my $submit = qualify_to_ref('submit', $subclass); + # "wrap" submit with _pre_submit only once + unless ( $Presubmit_Added{$subclass} ) { + my $real_submit = $subclass->can('submit'); + + no warnings 'redefine'; + no strict 'refs'; - $self->{_child_submit} = \&$submit; - *{"${subclass}::submit"} = sub { - my $self = shift; - $self->_pre_submit(); - } + *{"${subclass}::submit"} = sub { + my $self = shift; + return unless $self->_pre_submit(@_); + return $real_submit->($self, @_); + } } return $self; @@ -78,33 +81,34 @@ sub _risk_detect { $risk_transaction->submit(); if ($risk_transaction->is_success()) { if ( $risk_transaction->fraud_score <= $self->maximum_fraud_score()) { - $self->{_child_submit}->($self); + return 1; } else { - $self->is_success(0); $self->error_message('Excessive risk from risk management'); } } else { $self->error_message('Error in risk detection stage: ' . $risk_transaction->error_message); - $self->is_success(0); } + $self->is_success(0); + return 0; } -sub _pre_submit{ +my @Fraud_Class_Path = qw(Business::OnlinePayment Business::FraudDetect); + +sub _pre_submit { my ($self) = @_; my $fraud_detection = $self->fraud_detect(); # early return if user does not want optional risk mgt - return $self->{_child_submit}->($self,@_) unless $fraud_detection && length $fraud_detection; + return 1 unless $fraud_detection; # Search for an appropriate FD module - foreach my $subclass ( q(Business::OnlinePayment::) . $fraud_detection, - q(Business::FraudDetect::) . $fraud_detection) { - + foreach my $fraud_class ( @Fraud_Class_Path ) { + my $subclass = $fraud_class . "::" . $fraud_detection; if (!defined(&$subclass)) { - eval "use $subclass"; + eval "use $subclass ()"; if ($@) { - Carp::croak("serious problem loading fraud_detection module ($@)") unless - $@ =~ m/^Can\'t locate/; + Carp::croak("error loading fraud_detection module ($@)") + unless ( $@ =~ m/^Can\'t locate/ ); } else { my $risk_tx = bless ( { processor => $fraud_detection } , $subclass ); $risk_tx->build_subs(keys %fields); @@ -116,7 +120,10 @@ sub _pre_submit{ } } } -}; + Carp::croak("Unable to locate fraud_detection module $fraud_detection" + . " in \@INC under Fraud_Class_Path (\@Fraud_Class_Path" + . " contains: @Fraud_Class_Path) (\@INC contains: @INC)"); +} sub content { my($self,%params) = @_; @@ -213,9 +220,9 @@ Business::OnlinePayment - Perl extension for online payment processing $transaction->submit(); if($transaction->is_success()) { - print "Card processed successfully: ".$transaction->authorization()."\n"; + print "Card processed successfully: ", $transaction->authorization(), "\n"; } else { - print "Card was rejected: ".$transaction->error_message()."\n"; + print "Card was rejected: ", $transaction->error_message(), "\n"; } =head1 DESCRIPTION @@ -243,13 +250,9 @@ little depending on the processor, so we have chosen to use a system which defines specific fields in the frontend which get mapped to the correct fields in the backend. The currently defined fields are: -=over 4 - -=item * type +=head3 PROCESSOR FIELDS -Transaction type, supported types are: -Visa, MasterCard, American Express, Discover, Check (not all -processors support all these transaction types). +=over 4 =item * login @@ -259,6 +262,19 @@ Your login name to use for authentication to the online processor. Your password to use for authentication to the online processor. +=back + +=head3 GENERAL TRANSACTION FIELDS + +=over 4 + +=item * type + +Transaction type, supported types are: CC (credit card), ECHECK +(electronic check) and LEC (phone bill billing). Deprecated types +are: Visa, MasterCard, American Express, Discover, Check (not all +processors support all these transaction types). + =item * action What to do with the transaction (currently available are: Normal @@ -279,38 +295,74 @@ and the like, just a floating point number. An invoice number, for your use and not normally required, many processors require this field to be a numeric only field. +=back + +=head3 CUSTOMER INFO FIELDS + +=over 4 + =item * customer_id A customer identifier, again not normally required. =item * name -The customers name, your processor may not require this. +The customer's name, your processor may not require this. + +=item * first_name + +=item * last_name + +The customer's first and last name as separate fields. + +=item * company + +The customer's company name, not normally required. =item * address -The customers address (your processor may not require this unless you +The customer's address (your processor may not require this unless you are requiring AVS Verification). =item * city -The customers city (your processor may not require this unless you are -requiring AVS Verification). +The customer's city (your processor may not require this unless you +are requiring AVS Verification). =item * state -The customers state (your processor may not require this unless you +The customer's state (your processor may not require this unless you are requiring AVS Verification). =item * zip -The customers zip code (your processor may not require this unless you -are requiring AVS Verification). +The customer's zip code (your processor may not require this unless +you are requiring AVS Verification). =item * country Customer's country. +=item * ship_first_name + +=item * ship_last_name + +=item * ship_company + +=item * ship_address + +=item * ship_city + +=item * ship_state + +=item * ship_zip + +=item * ship_country + +These shipping address fields may be accepted by your processor. +Refer to the description for the corresponding non-ship field for +general information on each field. + =item * phone Customer's phone number. @@ -323,28 +375,87 @@ Customer's fax number. Customer's email address. +=item * customer_ip + +IP Address from which the transaction originated. + +=back + +=head3 CREDIT CARD FIELDS + +=over 4 + =item * card_number Credit card number (obviously not required for non-credit card transactions). -=item * exp_date +=item * cvv2 + +CVV2 number (also called CVC2 or CID) is a three- or four-digit +security code used to reduce credit card fraud. + +=item * expiration Credit card expiration (obviously not required for non-credit card transactions). +=item * recurring billing + +Recurring billing flag + +=back + +=head3 ELECTRONIC CHECK FIELDS + +=over 4 + =item * account_number -Bank account number for electronic checks or electronic funds transfer. +Bank account number for electronic checks or electronic funds +transfer. =item * routing_code -Bank's routing code for electronic checks or electronic funds transfer. +Bank's routing code for electronic checks or electronic funds +transfer. + +=item * account_type + +Account type for electronic checks or electronic funds transfer. + +=item * account_name + +Account holder's name for electronic checks or electronic funds +transfer. =item * bank_name Bank's name for electronic checks or electronic funds transfer. +=item * check_type + +Check type for electronic checks or electronic funds transfer. + +=item * customer_org + +Customer organization type. + +=item * customer_ssn + +Customer's social security number. Typically only required for +electronic checks or electronic funds transfer. + +=item * license_num + +Customer's driver's license number. Typically only required for +electronic checks or electronic funds transfer. + +=item * license_dob + +Customer's date of birth. Typically only required for electronic +checks or electronic funds transfer. + =back =head2 submit(); @@ -358,11 +469,11 @@ it failed (or undef if it has not been submitted yet). =head2 failure_status(); -If the transaction failed, it can optionally return a specific -failure status (normalized, not gateway-specific). Currently defined -statuses are: "expired", "nsf" (non-sufficient funds), "stolen", -"pickup", "blacklisted" and "declined" (card/transaction declines -only, not other errors). +If the transaction failed, it can optionally return a specific failure +status (normalized, not gateway-specific). Currently defined statuses +are: "expired", "nsf" (non-sufficient funds), "stolen", "pickup", +"blacklisted" and "declined" (card/transaction declines only, not +other errors). Note that (as of Aug 2006) this is only supported by some of the newest processor modules, and that, even if supported, a failure @@ -391,7 +502,7 @@ verification (if the processor supports it). =head2 transaction_type(); -Retrieve the transaction type (the 'type' argument to contents();). +Retrieve the transaction type (the 'type' argument to contents()). Generally only used internally, but provided in case it is useful. =head2 error_message(); @@ -412,11 +523,13 @@ YOUR OWN RISK). =head2 port(); -Retrieve or change the processor submission port (CHANGE AT YOUR OWN RISK). +Retrieve or change the processor submission port (CHANGE AT YOUR OWN +RISK). =head2 path(); -Retrieve or change the processor submission path (CHANGE AT YOUR OWN RISK). +Retrieve or change the processor submission path (CHANGE AT YOUR OWN +RISK). =head1 AUTHORS @@ -426,6 +539,11 @@ Jason Kohles, email@jasonkohles.com Phil Lobbes Ephil at perkpartners dot comE +=head1 MAILING LIST + +Please direct current development questions, patches, etc. to the mailing list: +http://420.am/cgi-bin/mailman/listinfo/bop-devel/ + =head1 DISCLAIMER THIS SOFTWARE IS PROVIDED "AS IS" AND WITHOUT ANY EXPRESS OR IMPLIED