@@ -, +, @@ --- C4/Reserves.pm | 2 +- Koha/Hold.pm | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) --- a/C4/Reserves.pm +++ a/C4/Reserves.pm @@ -1020,7 +1020,7 @@ sub ModReserveFill { } ); - # FIXME Must call Koha::Hold->cancel ? + # FIXME Must call Koha::Hold->cancel ? => No, should call ->filled and add the correct log Koha::Old::Hold->new( $hold->unblessed() )->store(); $hold->delete(); --- a/Koha/Hold.pm +++ a/Koha/Hold.pm @@ -316,7 +316,7 @@ sub cancel { $self->cancellationdate(dt_from_string); $self->priority(0); $self->_move_to_old; - $self->delete; + $self->SUPER::delete(); # Do not add a DELETE log # now fix the priority on the others.... C4::Reserves::_FixPriority({ biblionumber => $self->biblionumber }); --