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