Bugzilla – Attachment 151699 Details for
Bug 17506
Offline circ needs to be logged
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 17506: Offline circ needs to be logged
Bug-17506-Offline-circ-needs-to-be-logged.patch (text/plain), 1.62 KB, created by
Kyle M Hall (khall)
on 2023-05-25 15:48:59 UTC
(
hide
)
Description:
Bug 17506: Offline circ needs to be logged
Filename:
MIME Type:
Creator:
Kyle M Hall (khall)
Created:
2023-05-25 15:48:59 UTC
Size:
1.62 KB
patch
obsolete
>From 30f8021e40ff4e63dc33dab7c66c525f81e5422c Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Tue, 5 Apr 2022 09:07:01 -0400 >Subject: [PATCH] Bug 17506: Offline circ needs to be logged > >Returns via offline circ are not logged. > >Test Plan: >1) Via offline circ > * Return some items checked out to patrons > * Check out some items that are currently checked out to other > patrons >2) Apply the offline circs to Koha >3) Note the checkouts are logged, but not the returns >4) Apply this patch, restart all the things! >5) Repeat steps 1 and 2 >6) Note the checkins are now logged both for returns and for checkouts > where the item was checked out to a different patron >--- > C4/Circulation.pm | 6 ++++++ > 1 file changed, 6 insertions(+) > >diff --git a/C4/Circulation.pm b/C4/Circulation.pm >index 74d23db32c..9b4b3cdf80 100644 >--- a/C4/Circulation.pm >+++ b/C4/Circulation.pm >@@ -4082,6 +4082,9 @@ sub ProcessOfflineReturn { > $item->renewals_count(0); > $item->onloan(undef); > $item->store({ log_action => 0 }); >+ >+ logaction("CIRCULATION", "RETURN", $issue->{borrowernumber}, $itemnumber) >+ if C4::Context->preference("ReturnLog"); > return "Success."; > } else { > return "Item not issued."; >@@ -4110,6 +4113,9 @@ sub ProcessOfflineIssue { > $itemnumber, > $operation->{timestamp}, > ); >+ >+ logaction("CIRCULATION", "ISSUE", $issue->{borrowernumber}, $itemnumber) >+ if C4::Context->preference("ReturnLog"); > } > AddIssue( > $patron->unblessed, >-- >2.30.2
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 17506
:
132971
|
147842
|
151697
|
151698
|
151699
|
151707
|
151708
|
176184
|
176185