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
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
Status is NEW, but patches attached.
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.
I believe maybe instead of a --silent we should have a --verbose?
(In reply to Katrin Fischer from comment #5) > I believe maybe instead of a --silent we should have a --verbose? +1
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.
(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 :)