Bug 13197

Summary: Export.pl errors out on command line if --id_list_file is not provided
Product: Koha Reporter: Kyle M Hall <kyle>
Component: ToolsAssignee: Kyle M Hall <kyle>
Status: RESOLVED DUPLICATE QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: chris, katrin.fischer
Version: unspecified   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 13197 - Export.pl errors out on command line if --id_list_file is not provided
Bug 13197 - Export.pl errors out on command line if --id_list_file is not provided
Bug 13197 - Export.pl errors out on command line if --id_list_file is not provided

Description Kyle M Hall 2014-11-04 18:31:28 UTC
If one attempts to run export.pl from the command line without the --id_list_file switch, it errors out with the following message:

cannot open 0: No such file or directory at /usr/share/koha/intranet/cgi-bin/tools/export.pl line 209.
Comment 1 Kyle M Hall 2014-11-04 18:33:15 UTC Comment hidden (obsolete)
Comment 2 Kyle M Hall 2014-11-04 18:35:52 UTC Comment hidden (obsolete)
Comment 3 Chris Cormack 2014-11-05 06:42:20 UTC
Created attachment 33218 [details] [review]
Bug 13197 - Export.pl errors out on command line if --id_list_file is not provided

If one attempts to run export.pl from the command line without the
--id_list_file switch, it errors out with the following message:

cannot open 0: No such file or directory at
/usr/share/koha/intranet/cgi-bin/tools/export.pl line 209.

Test Plan:
1) Run tools/export.pl from the command line without the --id_list_file option
2) Note the error
3) Apply this patch
4) Repeat step 1
5) Note the error does not occur

Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Comment 4 Katrin Fischer 2014-11-05 21:48:20 UTC
Is it possible that this is also fixed by bug 13053 (PQA)? 

I needed to apply 13053 because I had an error message about missing permissions that it fixed, but then I could run the script without the parameter and no errors were shown.
Comment 5 Kyle M Hall 2014-11-14 12:58:27 UTC
It looks like I jumped the gun with this. It was already fixed in master when I posted this patch by bug 13085.

*** This bug has been marked as a duplicate of bug 13085 ***