check for zip and unzip commands, use multi-arg version of system to prevent the...

[freeside.git] / FS / bin / freeside-paymentech-download

diff --git a/FS/bin/freeside-paymentech-download b/FS/bin/freeside-paymentech-download
index 214b7f8..f4e0391 100755 (executable)
--- a/FS/bin/freeside-paymentech-download
+++ b/FS/bin/freeside-paymentech-download
@@ -17,7 +17,7 @@ getopts('vta:');
 #$Net::SFTP::Foreign::debug = -1;
 sub usage { "
   Usage:
-      paymentech-download [ -v ] [ -t ] [ -a archivedir ] user\n
+      freeside-paymentech-download [ -v ] [ -t ] [ -a archivedir ] user\n
 " }
 
 my $user = shift or die &usage;
@@ -30,6 +30,8 @@ if ( $opt_a ) {
     unless -w $opt_a;
 }
 
+my $unzip_check = `which unzip` or die "can't find unzip executable\n";
+
 #my $tmpdir = File::Temp->newdir();
 my $tmpdir = tempdir( CLEANUP => 1 ); #DIR=>somewhere?
 
@@ -64,9 +66,10 @@ BATCH: foreach my $filename (@files) {
   }
 
   #unzip file
-  system("unzip -P $password -q $tmpdir/${filename}_resp.zip -d $tmpdir");
+  system('unzip', '-P', '$password', '-q',
+           "$tmpdir/${filename}_resp.zip", '-d', $tmpdir);
   if(! -f "$tmpdir/${filename}_resp.xml") {
-    warn "failed to extract ${filename}_resp.xml\n";
+    warn "failed to extract ${filename}_resp.xml from ${filename}_resp.zip\n";
     next BATCH;
   }
 
@@ -100,9 +103,7 @@ print STDERR "Finished!\n" if $opt_v;
 
 =head1 NAME
 
-paymentech-download
-
-paymentech-download - Retrieve payment batch responses from Chase Paymentech.
+freeside-paymentech-download - Retrieve payment batch responses from Chase Paymentech.
 
 =head1 SYNOPSIS