--- a/C4/Reserves.pm +++ a/C4/Reserves.pm @@ -1125,7 +1125,15 @@ sub ModReserveFill { if C4::Context->preference('HoldsLog'); # FIXME Must call Koha::Hold->cancel ? => No, should call ->filled and add the correct log - Koha::Old::Hold->new( $hold->unblessed() )->store(); + my $old_hold = Koha::Old::Hold->new( $hold->unblessed() )->store(); + + Koha::Plugins->call( + 'after_hold_action', + { + action => 'fill', + payload => { hold => $old_hold->get_from_storage } + } + ); $hold->delete(); --- a/Koha/Hold.pm +++ a/Koha/Hold.pm @@ -34,6 +34,7 @@ use Koha::Items; use Koha::Libraries; use Koha::Old::Holds; use Koha::Calendar; +use Koha::Plugins; use Koha::Exceptions::Hold; @@ -112,6 +113,14 @@ sub suspend_hold { $self->suspend_until($date); $self->store(); + Koha::Plugins->call( + 'after_hold_action', + { + action => 'suspend', + payload => { hold => $self->get_from_storage } + } + ); + logaction( 'HOLDS', 'SUSPEND', $self->reserve_id, $self ) if C4::Context->preference('HoldsLog'); @@ -132,6 +141,14 @@ sub resume { $self->store(); + Koha::Plugins->call( + 'after_hold_action', + { + action => 'resume', + payload => { hold => $self->get_from_storage } + } + ); + logaction( 'HOLDS', 'RESUME', $self->reserve_id, $self ) if C4::Context->preference('HoldsLog'); @@ -545,7 +562,16 @@ sub cancel { } } - $self->_move_to_old; + my $old_me = $self->_move_to_old; + + Koha::Plugins->call( + 'after_hold_action', + { + action => 'cancel', + payload => { hold => $old_me->get_from_storage } + } + ); + $self->SUPER::delete(); # Do not add a DELETE log # now fix the priority on the others.... --