Bugzilla – Attachment 176184 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.65 KB, created by
Nick Clemens (kidclamp)
on 2025-01-07 12:19:09 UTC
(
hide
)
Description:
Bug 17506: Offline circ needs to be logged
Filename:
MIME Type:
Creator:
Nick Clemens (kidclamp)
Created:
2025-01-07 12:19:09 UTC
Size:
1.65 KB
patch
obsolete
>From 1915200c3ab5631385bcfc27d604e180252ee784 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 > >Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org> >--- > C4/Circulation.pm | 6 ++++++ > 1 file changed, 6 insertions(+) > >diff --git a/C4/Circulation.pm b/C4/Circulation.pm >index b9b66974bec..0cf848b3520 100644 >--- a/C4/Circulation.pm >+++ b/C4/Circulation.pm >@@ -4267,6 +4267,9 @@ sub ProcessOfflineReturn { > ); > $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."; >@@ -4296,6 +4299,9 @@ sub ProcessOfflineIssue { > $itemnumber, > $operation->{timestamp}, > ); >+ >+ logaction("CIRCULATION", "ISSUE", $issue->{borrowernumber}, $itemnumber) >+ if C4::Context->preference("ReturnLog"); > } > AddIssue( > $patron, >-- >2.39.5
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