Bug 15486: (follow-up) Default to unlimited
authorTomas Cohen Arazi <tomascohen@theke.io>
Mon, 24 Sep 2018 15:56:07 +0000 (12:56 -0300)
committerNick Clemens <nick@bywatersolutions.com>
Wed, 24 Oct 2018 13:24:06 +0000 (13:24 +0000)
While it doesn't have a practical effect (the variable is reset several
lines below) I agree this should be explicitly set to its default.

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Signed-off-by: Séverine QUEUNE <severine.queune@bulac.fr>
Signed-off-by: Josef Moravec <josef.moravec@gmail.com>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
C4/Reserves.pm

index 3c3e158..78e8f6a 100644 (file)
@@ -314,7 +314,7 @@ sub CanItemBeReserved {
     my $ruleitemtype;    # itemtype of the matching issuing rule
     my $allowedreserves  = 0; # Total number of holds allowed across all records
     my $holds_per_record = 1; # Total number of holds allowed for this one given record
-    my $holds_per_day    = 0; # Total number of holds allowed per day for the given patron
+    my $holds_per_day;        # Default to unlimited
 
     # we retrieve borrowers and items informations #
     # item->{itype} will come for biblioitems if necessery