Bug 10810 - Fix synopsis for -html option to overdue_notices.pl
authorMagnus Enger <magnus@enger.priv.no>
Sat, 31 Aug 2013 13:33:24 +0000 (15:33 +0200)
committerTomas Cohen Arazi <tomascohen@gmail.com>
Wed, 6 Nov 2013 16:36:57 +0000 (13:36 -0300)
commit63fa18513fceb8fee6b6a35d885ec50b0d55c969
treef895e96a98db6ce03872b5cc38e1fe42f43777ec
parent4bc27151a1cb5a50825fc739e8ccf84aef913afb
Bug 10810 - Fix synopsis for -html option to overdue_notices.pl

The synopsis for the -html option used to be:
  -html <filename> Output html to file
but the argument to this option should actually be a directory,
not a filename. This patch fixes the synopsis.

To test:
- Run perldoc misc/cronjobs/overdue_notices.pl
- Check that the synopsis is as shown above
- Apply this patch
- Run perldoc misc/cronjobs/overdue_notices.pl
- Check that
  - the synopsis asks for a directory, not a filename
  - that the description of the -html option further down in the
    perldoc now mentions the filename that the HTML file will be
    created with.

Signed-off-by: Bernardo Gonzalez Kriegel <bgkriegel@gmail.com>
Works as described. No koha-qa errors.

If feeded with a filename -html option fails with many messages like
print() on closed filehandle $html_fh at misc/cronjobs/overdue_notices.pl line 384.
Perhaps it could be improved, but that was the old behavior.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Improves documentation, no negative side effects found.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
(cherry picked from commit b54e33c1f62e4aa2c7e46a7fafa445f33f570326)
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
misc/cronjobs/overdue_notices.pl