gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 40405: Set decreaseLoanHighHolds\* to 0 

When reviewing the different sysprefs that had "NULL" as default, I
only found those 2 occurrences that could be problematic. I think we
should default to 0 here.

Signed-off-by: Martin Renvoize <martin.renvoize@openfifth.co.uk>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit ca6cbc3cf9)
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
This commit is contained in:
Jonathan Druart 2025-07-16 11:59:17 +02:00 committed by Paul Derscheid
parent baf8af8c99
commit 5d5ccb8073
No known key found for this signature in database
GPG Key ID: C25F9FD44CED595B
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
C4/Circulation.pm
View File

@ -1457,7 +1457,7 @@ sub checkHighHolds {
$return_data->{outstanding} = $holds->count();
my $decreaseLoanHighHoldsControl = C4::Context->preference('decreaseLoanHighHoldsControl');
my $decreaseLoanHighHoldsValue = C4::Context->preference('decreaseLoanHighHoldsValue');
my $decreaseLoanHighHoldsValue = C4::Context->preference('decreaseLoanHighHoldsValue') || 0;
my $decreaseLoanHighHoldsIgnoreStatuses = C4::Context->preference('decreaseLoanHighHoldsIgnoreStatuses');
my @decreaseLoanHighHoldsIgnoreStatuses = split( /,/, $decreaseLoanHighHoldsIgnoreStatuses );
@ -1529,7 +1529,7 @@ sub checkHighHolds {
# overrides decreaseLoanHighHoldsDuration syspref
$duration = $rule;
} else {
$duration = C4::Context->preference('decreaseLoanHighHoldsDuration');
$duration = C4::Context->preference('decreaseLoanHighHoldsDuration') || 0;
}
my $reduced_datedue = $calendar->addDuration( $issuedate, $duration );
$reduced_datedue->set_hour( $orig_due->hour );