diff options
| author | Jonathan Prykop <jonathan@freeside.biz> | 2015-12-22 00:28:53 -0600 |
|---|---|---|
| committer | Jonathan Prykop <jonathan@freeside.biz> | 2015-12-22 00:28:53 -0600 |
| commit | 1259a17db297fa2352619b29f2c5bd34e313cd64 (patch) | |
| tree | 644f9c30e6eb1a470f671afb8a3f9971d6bb7302 /httemplate | |
| parent | 38444ef88b5e93aa9aa724369ae8fe17c97fa480 (diff) | |
RT#34295: Error when attempting to create batch payments [better handling of empty batches]
Diffstat (limited to 'httemplate')
| -rw-r--r-- | httemplate/misc/download-batch.cgi | 13 |
1 files changed, 12 insertions, 1 deletions
diff --git a/httemplate/misc/download-batch.cgi b/httemplate/misc/download-batch.cgi index f3a31eb3b..c4bc37e93 100644 --- a/httemplate/misc/download-batch.cgi +++ b/httemplate/misc/download-batch.cgi @@ -1,4 +1,4 @@ -<% $pay_batch->export_batch(%opt) %><%init> +<% $exporttext %><%init> #http_header('Content-Type' => 'text/comma-separated-values' ); #IE chokes http_header('Content-Type' => 'text/plain' ); # not necessarily correct... @@ -23,4 +23,15 @@ elsif ( $cgi->param('format') =~ /^([\w\- ]+)$/ ) { my $pay_batch = qsearchs('pay_batch', { batchnum => $batchnum } ); die "Batch not found: '$batchnum'" if !$pay_batch; +my $exporttext = $pay_batch->export_batch(%opt); +unless ($exporttext) { + http_header('Content-Type' => 'text/html' ); + $exporttext = <<EOF; +<SCRIPT> +alert('Batch was empty, and has been resolved'); +window.top.location.href = '${p}search/pay_batch.cgi?magic=_date;open=1;intransit=1;resolved=1'; +</SCRIPT> +EOF +} + </%init> |