@@ -, +, @@ --- C4/Circulation.pm | 1 - Koha/Hold.pm | 2 +- Koha/Item/Transfer.pm | 3 +-- 3 files changed, 2 insertions(+), 4 deletions(-) --- a/C4/Circulation.pm +++ a/C4/Circulation.pm @@ -3446,7 +3446,6 @@ sub GetTransfers { FROM branchtransfers WHERE itemnumber = ? AND datearrived IS NULL - AND datecancelled IS NULL '; my $sth = $dbh->prepare($query); $sth->execute($itemnumber); --- a/Koha/Hold.pm +++ a/Koha/Hold.pm @@ -519,7 +519,7 @@ sub cancel { # NOTE: Transfers are well bound with Holds.. as such we have to check that there is actually a # transfer enqueued and in transit here prior to trying to cancel it. if ( $transfer && $transfer->in_transit ) { - $transfer->cancel({ reason => 'CancelReserve', force => 1 }); + $transfer->cancel({ reason => 'CancelReserve', force => 1, replace => 1 }); } } $self->cancellationdate( dt_from_string->strftime( '%Y-%m-%d %H:%M:%S' ) ); --- a/Koha/Item/Transfer.pm +++ a/Koha/Item/Transfer.pm @@ -110,8 +110,7 @@ sub in_transit { my ($self) = @_; return ( defined( $self->datesent ) - && !defined( $self->datearrived ) - && !defined( $self->datecancelled ) ); + && !defined( $self->datearrived ) ); } =head3 receive --