Bug 13085: bugfix to allow call of export.pl in commandline without id_list_file...
authorStéphane Delaune <stephane.delaune@biblibre.com>
Wed, 15 Oct 2014 13:23:35 +0000 (15:23 +0200)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Wed, 22 Oct 2014 12:31:30 +0000 (09:31 -0300)
Test plan
1/ run ./tools/export.pl
2/ notice an error like cannot open 0: No such file or directory at ./tools/export.pl line 209.
3/ apply patch
4/ run again
5/ no errors

Signed-off-by: Chris <chris@bigballofwax.co.nz>
Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, adding an additional check.

Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
tools/export.pl

index 8a5c2a5..97415f8 100755 (executable)
@@ -206,7 +206,7 @@ if ( $op eq "export" ) {
         my $authtype        = $query->param('authtype');
         my $filefh;
         if ($commandline) {
-            open $filefh,"<", $id_list_file or die "cannot open $id_list_file: $!";
+            open $filefh,"<", $id_list_file or die "cannot open $id_list_file: $!" if $id_list_file;
         } else {
             $filefh = $query->upload("id_list_file");
         }