Attachment 81340 Details for Bug 21698 – Bug 21698 - FIX POD of cancel_unfilled_holds.pl

[patch] Bug 21698 - FIX POD of cancel_unfilled_holds.pl

Bug-21698---FIX-POD-of-cancelunfilledholdspl.patch (text/plain), 2.66 KB, created by Magnus Enger on 2018-10-26 20:56:24 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Magnus Enger

Created: 2018-10-26 20:56:24 UTC

Size: 2.66 KB

patch

obsolete

>From f94d413e733f56be90aafb953d4babf4b971cd21 Mon Sep 17 00:00:00 2001
>From: Magnus Enger <magnus@libriotech.no>
>Date: Fri, 26 Oct 2018 22:47:16 +0200
>Subject: [PATCH] Bug 21698 - FIX POD of cancel_unfilled_holds.pl
>
>To test:
>- Run "perldoc misc/cronjobs/holds/cancel_unfilled_holds.pl"
>- Notice some problems:
>-- No description in the NAME section
>-- Descriptions of options in the SYNOPSIS section
>-- A literal head1
>-- Long options are listed with a single leading dash
>-- Options --verbose and --confirm are not listed under OPTIONS)
>- Take a look at the script itself to see that --verbose and
>  --confirm should be listed.
>- Also note the typo on line 97: 'v|verbosev' => \$verbose,
>- Apply the patch
>- Look at the script and perldoc again and check that all problems
>  described above are now fixed.
>---
> misc/cronjobs/holds/cancel_unfilled_holds.pl | 36 +++++++++-----------
> 1 file changed, 17 insertions(+), 19 deletions(-)
>
>diff --git a/misc/cronjobs/holds/cancel_unfilled_holds.pl b/misc/cronjobs/holds/cancel_unfilled_holds.pl
>index 407bc9cc8e..bea32fff7c 100755
>--- a/misc/cronjobs/holds/cancel_unfilled_holds.pl
>+++ b/misc/cronjobs/holds/cancel_unfilled_holds.pl
>@@ -39,45 +39,43 @@ cronlogaction();
> 
> =head1 NAME
> 
>-cancel_unfilled_holds.pl
>+cancel_unfilled_holds.pl - script to delete unfilled holds after a given 
>+number of days.
> 
> =head1 SYNOPSIS
> 
>-cancel_unfilled_holds.pl
>-    [-days][-library][-holidays]
>+ cancel_unfilled_holds.pl [--days][--library][--holidays][--confirm][--verbose]
> 
>- Options:
>-    -help                       brief help
>-    -days                       cancel holds placed this many days ago which have not been filled
>-    -library                    [repeatable] limit to specified branch(es)
>-    -holidays                   skip holidays when calculating days waiting
>-    -v                          verbose
>-
>-head1 OPTIONS
>+=head1 OPTIONS
> 
> =over 8
> 
>-=item B<-help>
>+=item B<--help | -h>
> 
> Print brief help and exit.
> 
>-=item B<-man>
>-
>-Print full documentation and exit.
>-
>-=item B<-days>
>+=item B<--days>
> 
> Specify the number of days waiting since a hold that remains unfilled was placed.
> E.g. a value of 730 would cancel holds placed 2 years ago or more that have never been filled
> 
>-=item B<-library>
>+=item B<--library>
> 
> Repeatable option to specify which branchcode(s) to cancel holds for.
> 
>-=item B<-holidays>
>+=item B<--holidays>
> 
> This switch specifies whether to count holidays as days waiting. Default is no.
> 
>+=item B<--confirm>
>+
>+Without this option, the script will run in test mode, and only report what it
>+would have done if it were not running in test mode.
>+
>+=item B<--verbose | -v>
>+
>+More verbose output.
>+
> =back
> 
> =cut
>-- 
>2.17.1

Actions: View | Diff | Splinter Review

Attachments on bug 21698: 81340 | 81499 | 81534 | 81535