View | Details | Raw Unified | Return to bug 22005
Collapse All | Expand All

(-)a/C4/Circulation.pm (-4 / +7 lines)
Lines 1205-1212 sub checkHighHolds { Link Here
1205
1205
1206
            # static means just more than a given number of holds on the record
1206
            # static means just more than a given number of holds on the record
1207
1207
1208
            # If the number of holds is less than the threshold, we can stop here
1208
            # If the number of holds is less than or equal to the threshold, we can stop here
1209
            if ( $holds->count() < $decreaseLoanHighHoldsValue ) {
1209
            # The sysprefs say: "[Enable] the reduction of loan period to [x] days for high
1210
            # demand items with *more than* [y] holds [on the record]."
1211
            # This must mean that if the number of holds and the value of
1212
            # decreaseLoanHighHoldsValue are equal, checkHighHolds should NOT be triggered
1213
            if ( $holds->count() <= $decreaseLoanHighHoldsValue ) {
1210
                return $return_data;
1214
                return $return_data;
1211
            }
1215
            }
1212
        }
1216
        }
Lines 1229-1235 sub checkHighHolds { Link Here
1229
1233
1230
            my $threshold = $items_count + $decreaseLoanHighHoldsValue;
1234
            my $threshold = $items_count + $decreaseLoanHighHoldsValue;
1231
1235
1232
            # If the number of holds is less than the count of items we have
1236
            # If the number of holds is less than or equal to the count of items we have
1233
            # plus the number of holds allowed above that count, we can stop here
1237
            # plus the number of holds allowed above that count, we can stop here
1234
            if ( $holds->count() <= $threshold ) {
1238
            if ( $holds->count() <= $threshold ) {
1235
                return $return_data;
1239
                return $return_data;
1236
- 

Return to bug 22005