Created attachment 8632 [details] [review] 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.
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
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>
QA Comment: Hope you got them all ;) Passed QA.
There have been no further reports of problems so I am marking this bug resolved.