Bugzilla – Attachment 28279 Details for
Bug 11856
Minor improvements on code documentation (omnibus)
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 11856: Add confirm option to POD in advance_notices.pl
Bug-11856-Add-confirm-option-to-POD-in-advancenoti.patch (text/plain), 1.84 KB, created by
Jonathan Druart
on 2014-05-15 13:18:21 UTC
(
hide
)
Description:
Bug 11856: Add confirm option to POD in advance_notices.pl
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2014-05-15 13:18:21 UTC
Size:
1.84 KB
patch
obsolete
>From 5cb86674f9dfdcec1429c9f83800d58d7cdf2b1c Mon Sep 17 00:00:00 2001 >From: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> >Date: Thu, 17 Apr 2014 11:59:55 +0200 >Subject: [PATCH] Bug 11856: Add confirm option to POD in advance_notices.pl >MIME-Version: 1.0 >Content-Type: text/plain; charset=UTF-8 >Content-Transfer-Encoding: 8bit > >Also removes a commented line that changed the value of $confirm. >If it would default to 1, we do not need it at all. >I am not saying that we need it, but just documenting that we have it. > >Small change in documentation. >Signed-off-by: Marc Véron <veron@veron.ch> >--- > misc/cronjobs/advance_notices.pl | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > >diff --git a/misc/cronjobs/advance_notices.pl b/misc/cronjobs/advance_notices.pl >index 405b0e1..c7c2e0f 100755 >--- a/misc/cronjobs/advance_notices.pl >+++ b/misc/cronjobs/advance_notices.pl >@@ -91,7 +91,10 @@ the patrons are printed to standard out. > > Defines the maximum number of days in advance to send advance notices. > >+=item B<-c> > >+Confirm flag: Add this option. The script will only print a usage >+statement otherwise. > > =item B<--itemscontent> > >@@ -160,7 +163,6 @@ messages to patrons when their messages are overdue. > > # These are defaults for command line options. > my $confirm; # -c: Confirm that the user has read and configured this script. >-# my $confirm = 1; # -c: Confirm that the user has read and configured this script. > my $nomail; # -n: No mail. Will not send any emails. > my $mindays = 0; # -m: Maximum number of days in advance to send notices > my $maxdays = 30; # -e: the End of the time period >-- >1.7.10.4
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 11856
:
27232
|
28139
|
28279
|
28280