From af142b9ba5d27453f98c47908d8ad0bd68cacdff Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 30 Jan 2020 10:29:05 -0500 Subject: [PATCH] Bug 24547: Add more action logs for holds It seems like ModReserveFill and ModReserveAffect should both produce action logs for holds. Test Plan: 1) Apply this patch 2) Place a hold 3) Check in the item to trap the hold 4) Check out the item to fill the hold 5) Check the action logs for that reserve id 6) Note the new logs! Signed-off-by: David Nind --- C4/Reserves.pm | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/C4/Reserves.pm b/C4/Reserves.pm index f2bf8bd5d3..ea12f279f5 100644 --- a/C4/Reserves.pm +++ b/C4/Reserves.pm @@ -1026,6 +1026,9 @@ sub ModReserveFill { } ); + logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, Dumper($hold->unblessed) ) + 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(); @@ -1126,6 +1129,9 @@ sub ModReserveAffect { CartToShelf( $itemnumber ); } + logaction( 'HOLDS', 'MODIFY', $hold->reserve_id, Dumper($hold->unblessed) ) + if C4::Context->preference('HoldsLog'); + return; } -- 2.11.0