Bugzilla – Attachment 98215 Details for
Bug 24547
Add more action logs for holds
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 24547: Add more action logs for holds
Bug-24547-Add-more-action-logs-for-holds.patch (text/plain), 1.31 KB, created by
David Nind
on 2020-01-31 08:01:43 UTC
(
hide
)
Description:
Bug 24547: Add more action logs for holds
Filename:
MIME Type:
Creator:
David Nind
Created:
2020-01-31 08:01:43 UTC
Size:
1.31 KB
patch
obsolete
>From af142b9ba5d27453f98c47908d8ad0bd68cacdff Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >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 <david@davidnind.com> >--- > 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
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 24547
:
98174
|
98215
|
102565
|
102569