projects
/
freeside.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
discounts, RT#6679
[freeside.git]
/
FS
/
FS
/
pay_batch.pm
diff --git
a/FS/FS/pay_batch.pm
b/FS/FS/pay_batch.pm
index
e0531f4
..
59ff2c3
100644
(file)
--- a/
FS/FS/pay_batch.pm
+++ b/
FS/FS/pay_batch.pm
@@
-202,6
+202,7
@@
sub import_results {
my $begin_condition = $info->{'begin_condition'};
my $end_condition = $info->{'end_condition'};
my $end_hook = $info->{'end_hook'};
my $begin_condition = $info->{'begin_condition'};
my $end_condition = $info->{'end_condition'};
my $end_hook = $info->{'end_hook'};
+ my $skip_condition = $info->{'skip_condition'};
my $hook = $info->{'hook'};
my $approved_condition = $info->{'approved'};
my $declined_condition = $info->{'declined'};
my $hook = $info->{'hook'};
my $approved_condition = $info->{'approved'};
my $declined_condition = $info->{'declined'};
@@
-295,8
+296,13
@@
sub import_results {
$hash{$field} = $value;
}
$hash{$field} = $value;
}
- if ( defined($begin_condition) and &{$begin_condition}(\%hash, $line)) {
- undef $begin_condition;
+ if ( defined($begin_condition) ) {
+ if ( &{$begin_condition}(\%hash, $line) ) {
+ undef $begin_condition;
+ }
+ else {
+ next;
+ }
}
if ( defined($end_condition) and &{$end_condition}(\%hash, $line) ) {
}
if ( defined($end_condition) and &{$end_condition}(\%hash, $line) ) {
@@
-309,6
+315,10
@@
sub import_results {
last;
}
last;
}
+ if ( defined($skip_condition) and &{$skip_condition}(\%hash, $line) ) {
+ next;
+ }
+
my $cust_pay_batch =
qsearchs('cust_pay_batch', { 'paybatchnum' => $hash{'paybatchnum'}+0 } );
unless ( $cust_pay_batch ) {
my $cust_pay_batch =
qsearchs('cust_pay_batch', { 'paybatchnum' => $hash{'paybatchnum'}+0 } );
unless ( $cust_pay_batch ) {
@@
-393,8
+403,8
@@
sub import_results {
}
}
-sub export_batch {
# Formerly httemplate/misc/download-batch.cgi
# Formerly httemplate/misc/download-batch.cgi
+sub export_batch {
my $self = shift;
my $conf = new FS::Conf;
my $format = shift || $conf->config('batch-default_format')
my $self = shift;
my $conf = new FS::Conf;
my $format = shift || $conf->config('batch-default_format')
@@
-402,26
+412,23
@@
sub export_batch {
my $info = $export_info{$format} or die "Format not found: '$format'\n";
&{$info->{'init'}}($conf) if exists($info->{'init'});
my $info = $export_info{$format} or die "Format not found: '$format'\n";
&{$info->{'init'}}($conf) if exists($info->{'init'});
+ my $curuser = $FS::CurrentUser::CurrentUser;
+
my $oldAutoCommit = $FS::UID::AutoCommit;
local $FS::UID::AutoCommit = 0;
my $dbh = dbh;
my $oldAutoCommit = $FS::UID::AutoCommit;
local $FS::UID::AutoCommit = 0;
my $dbh = dbh;
- my $error;
-
my $first_download;
my $first_download;
- if($self->status eq 'O') {
+ my $status = $self->status;
+ if ($status eq 'O') {
$first_download = 1;
$first_download = 1;
- }
- elsif($self->status eq 'I' and
-
$FS::CurrentUser::CurrentU
ser->access_right('Reprocess batches')) {
+ my $error = $self->set_status('I');
+ die "error updating pay_batch status: $error\n" if $error;
+
} elsif ($status eq 'I' && $curu
ser->access_right('Reprocess batches')) {
$first_download = 0;
$first_download = 0;
+ } else {
+ die "No pending batch.\n";
}
}
- else {
- die "No pending batch.\n"
- }
-
- $error = $self->set_status('I');
- die "error updating pay_batch status: $error\n" if $error;
my $batch = '';
my $batchtotal = 0;
my $batch = '';
my $batchtotal = 0;
@@
-440,7
+447,7
@@
sub export_batch {
foreach my $cust_pay_batch (@cust_pay_batch) {
if($first_download) {
my $balance = $cust_pay_batch->cust_main->balance;
foreach my $cust_pay_batch (@cust_pay_batch) {
if($first_download) {
my $balance = $cust_pay_batch->cust_main->balance;
- $error = '';
+
my
$error = '';
if($balance <= 0) { # then don't charge this customer
$error = $cust_pay_batch->delete;
undef $cust_pay_batch;
if($balance <= 0) { # then don't charge this customer
$error = $cust_pay_batch->delete;
undef $cust_pay_batch;
@@
-458,7
+465,7
@@
sub export_batch {
if($cust_pay_batch) { # that is, it wasn't deleted
$batchcount++;
$batchtotal += $cust_pay_batch->amount;
if($cust_pay_batch) { # that is, it wasn't deleted
$batchcount++;
$batchtotal += $cust_pay_batch->amount;
- $batch .= &{$info->{'row'}}($cust_pay_batch, $self) . "\n";
+ $batch .= &{$info->{'row'}}($cust_pay_batch, $self
, $batchcount, $batchtotal
) . "\n";
}
}
my $f = $info->{'footer'};
}
}
my $f = $info->{'footer'};
@@
-470,7
+477,7
@@
sub export_batch {
}
if ($info->{'autopost'}) {
}
if ($info->{'autopost'}) {
- $error = &{$info->{'autopost'}}($self, $batch);
+
my
$error = &{$info->{'autopost'}}($self, $batch);
if($error) {
$dbh->rollback or die $dbh->errstr if $oldAutoCommit;
die $error;
if($error) {
$dbh->rollback or die $dbh->errstr if $oldAutoCommit;
die $error;