diff options
author | ivan <ivan> | 2003-10-23 07:49:56 +0000 |
---|---|---|
committer | ivan <ivan> | 2003-10-23 07:49:56 +0000 |
commit | aa71df24486955d494c30ccc14fbe5917029ddec (patch) | |
tree | 8c974c42c7c106adfa33b93f92bf746caea93d55 /FS/FS/cust_pay_batch.pm | |
parent | e5b08c9a31a68826e00b90b62249a57a24ebb9e3 (diff) |
Declined results from batches should now suspend the relevant customer.
Diffstat (limited to 'FS/FS/cust_pay_batch.pm')
-rw-r--r-- | FS/FS/cust_pay_batch.pm | 69 |
1 files changed, 48 insertions, 21 deletions
diff --git a/FS/FS/cust_pay_batch.pm b/FS/FS/cust_pay_batch.pm index f53e55b..8059f1c 100644 --- a/FS/FS/cust_pay_batch.pm +++ b/FS/FS/cust_pay_batch.pm @@ -188,6 +188,18 @@ sub check { $self->SUPER::check; } +=item cust_main + +Returns the customer (see L<FS::cust_main>) for this batched credit card +payment. + +=cut + +sub cust_main { + my $self = shift; + qsearchs( 'cust_main', { 'custnum' => $self->custnum } ); +} + =back =head1 SUBROUTINES @@ -212,8 +224,9 @@ sub import_results { my @fields; my $end_condition; my $end_hook; - my $condition; my $hook; + my $approved_condition; + my $declined_condition; if ( $format eq 'csv-td_canada_trust-merchant_pc_batch' ) { @@ -252,11 +265,6 @@ sub import_results { ''; }; - $condition = sub { - my $hash = shift; - $hash->{'result'} == 3 && $hash->{'type'} eq '0'; - }; - $hook = sub { my $hash = shift; $hash->{'paid'} = sprintf("%.2f", $hash->{'paid'} / 100 ); @@ -268,6 +276,18 @@ sub import_results { substr($hash->{'_date'}, 0, 4)-1900, ); }; + $approved_condition = sub { + my $hash = shift; + $hash->{'type'} eq '0' && $hash->{'result'} == 3; + }; + + $declined_condition = sub { + my $hash = shift; + $hash->{'type'} eq '0' && ( $hash->{'result'} == 4 + || $hash->{'result'} == 5 ); + }; + + } else { return "Unknown format $format"; } @@ -327,24 +347,31 @@ sub import_results { return "error removing paybatchnum $hash{'paybatchnum'}: $error\n"; } - next unless &{$condition}(\%hash); - &{$hook}(\%hash); - my $cust_pay = new FS::cust_pay ( { - 'custnum' => $custnum, - 'payby' => 'CARD', - 'paybatch' => $paybatch, - map { $_ => $hash{$_} } (qw( paid _date payinfo )), - } ); - $error = $cust_pay->insert; - if ( $error ) { - $dbh->rollback if $oldAutoCommit; - return "error adding payment paybatchnum $hash{'paybatchnum'}: $error\n"; - } - $total += $hash{'paid'}; + if ( &{$approved_condition}(\%hash) ) { + + my $cust_pay = new FS::cust_pay ( { + 'custnum' => $custnum, + 'payby' => 'CARD', + 'paybatch' => $paybatch, + map { $_ => $hash{$_} } (qw( paid _date payinfo )), + } ); + $error = $cust_pay->insert; + if ( $error ) { + $dbh->rollback if $oldAutoCommit; + return "error adding payment paybatchnum $hash{'paybatchnum'}: $error\n"; + } + $total += $hash{'paid'}; + + $cust_pay->cust_main->apply_payments; - $cust_pay->cust_main->apply_payments; + } elsif ( &{$declined_condition}(\%hash) ) { + + #this should be configurable... if anybody else ever uses batches + $cust_pay_batch->cust_main->suspend; + + } } |