Bug 7829

Summary: reports/ remove all exit(1) for plack
Product: Koha Reporter: Dobrica Pavlinusic <dpavlin>
Component: ReportsAssignee: Dobrica Pavlinusic <dpavlin>
Status: CLOSED FIXED QA Contact: Marcel de Rooy <m.de.rooy>
Severity: enhancement    
Priority: P5 - low CC: alex.arnaud, m.de.rooy, paul.poulain
Version: 3.8   
Hardware: All   
OS: All   
See Also: http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7772
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=8833
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 7829 - reports/ remove all exit(1) for plack
[Signed-off] Bug 7829 - reports/ remove all exit(1) for plack

Description Dobrica Pavlinusic 2012-03-26 14:48:25 UTC

    
Comment 1 Dobrica Pavlinusic 2012-03-26 14:52:30 UTC Comment hidden (obsolete)
Comment 2 Alex Arnaud 2012-03-27 08:17:30 UTC
Having koha on plack, i tested making reports such as acquisitions_stats.pl, reserves_stats.pl, issues_stats.pl etc ... and got Internal Server Error on each script when submitting the form.

The patch fix this problem. So signing off
Comment 3 Alex Arnaud 2012-03-27 08:20:02 UTC
Created attachment 8644 [details] [review]
[Signed-off] Bug 7829 - reports/ remove all exit(1) for plack

In Bug 7772 Ian correctly noted that reports have exit(1) all over the place.
This is left over from old code, and this patch changes them to exit(0).

I decided to use plain exit as opposed to explicit exit(0) since it produces
cleaner code, but I'm welcoming suggestion on this.

Signed-off-by: Alex Arnaud <alex.arnaud@biblibre.com>
Comment 4 Marcel de Rooy 2012-03-28 11:33:25 UTC
QA Comment: Hope you got them all ;) Passed QA.
Comment 5 Jared Camins-Esakov 2012-12-31 00:49:22 UTC
There have been no further reports of problems so I am marking this bug resolved.