Bug 13811

Summary: Silent Cronjobs
Product: Koha Reporter: Maxime Beaulieu <maxime.beaulieu>
Component: Command-line UtilitiesAssignee: Bugs List <koha-bugs>
Status: Failed QA --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: caroline.cyr-la-rose, dcook, eric.begin, f.demians, jonathan.druart, marjorie.barry-vila, philippe.blouin, veron
Version: Main   
Hardware: All   
OS: All   
GIT URL: Change sponsored?: ---
Patch complexity: --- Documentation contact:
Documentation submission: Text to go in the release notes:
Version(s) released in:
Circulation function:
Attachments: Added -s|--silent option to borrowers-force-messaging-defaults
check-url's --html option prints directly to a file instead of stdout.

Description Maxime Beaulieu 2015-03-09 19:16:26 UTC

    
Comment 1 Maxime Beaulieu 2015-03-09 19:19:19 UTC
Created attachment 36749 [details] [review]
Added -s|--silent option to borrowers-force-messaging-defaults

The script will not print to stdout when the flag is activated.

Test:
    Run the script with the flag. All to stdout is eliminated.
    Run the scirpt without the flag. A report is printed to stdout.

sponsored by: CCSR
Comment 2 Maxime Beaulieu 2015-03-09 19:20:18 UTC
Created attachment 36750 [details] [review]
check-url's --html option prints directly to a file instead of stdout.

This change makes the option behave more like other cronjob's --html or --csv options.
It also makes the script silent while printing html, as nothing is printed to stdout unless the verbose option is passed.

The script's behaviour is not changed when launched without options. (Prints in stdout)

sponsored by: CCSR
Comment 3 Marc Véron 2015-03-21 06:14:32 UTC
Status is NEW, but patches attached.
Comment 4 Frédéric Demians 2015-04-13 14:09:27 UTC
1) Could you document the new --silent parameter in the POD section of
   borrowers-force-messaging-defaults?

2) Why don't you modify check-url-quick.pl? check-url.pl is supposed to be
   deprecated.

3) As 1), the new way --html operates imposes to modify the documentation.

4) It may be better to have two bugs: one for each modified script.
Comment 5 Katrin Fischer 2023-07-28 21:55:06 UTC
I believe maybe instead of a --silent we should have a --verbose?
Comment 6 David Cook 2023-07-30 23:48:45 UTC
(In reply to Katrin Fischer from comment #5)
> I believe maybe instead of a --silent we should have a --verbose?

+1
Comment 7 Blou 2023-08-07 18:24:14 UTC
ooooh, a time machine!  :-)

Yes, fair point.  I had to dig to recover references to this Bug.  Unfortunately, our sponsor pulled the plug on this one a while ago.  Could be simple to do, but I can't prioritize this in any fashion.
Comment 8 Katrin Fischer 2023-08-07 18:48:15 UTC
(In reply to Blou from comment #7)
> ooooh, a time machine!  :-)
> 
> Yes, fair point.  I had to dig to recover references to this Bug. 
> Unfortunately, our sponsor pulled the plug on this one a while ago.  Could
> be simple to do, but I can't prioritize this in any fashion.

That's too bad, but thanks for the feedback :)