@@ -, +, @@ --- C4/Reserves.pm | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) --- a/C4/Reserves.pm +++ a/C4/Reserves.pm @@ -1119,16 +1119,12 @@ sub ModReserve { $hold->cancel({ cancellation_reason => $cancellation_reason }); } elsif ($hold->found && $hold->priority eq '0' && $date) { - logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, $hold ) - if C4::Context->preference('HoldsLog'); - # The only column that can be updated for a found hold is the expiration date $hold->expirationdate($date)->store(); - } - elsif ($rank =~ /^\d+/ and $rank > 0) { logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, $hold ) if C4::Context->preference('HoldsLog'); - + } + elsif ($rank =~ /^\d+/ and $rank > 0) { my $properties = { priority => $rank, branchcode => $branchcode, @@ -1156,6 +1152,8 @@ sub ModReserve { } _FixPriority({ reserve_id => $reserve_id, rank =>$rank }); + logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, $hold ) + if C4::Context->preference('HoldsLog'); } } --