Bug 41163 - Circulation logs record issuing branch in database but show logged-in branch in log viewer
Summary: Circulation logs record issuing branch in database but show logged-in branch ...
Status: Pushed to main
Alias: None
Product: Koha
Classification: Unclassified
Component: Tools (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Andrew Fuerste-Henry
QA Contact: Lisette Scheer
URL:
Keywords:
: 41590 (view as bug list)
Depends on: 9762
Blocks:
  Show dependency treegraph
 
Reported: 2025-10-31 17:53 UTC by Andrew Fuerste-Henry
Modified: 2026-01-28 21:26 UTC (History)
2 users (show)

See Also:
GIT URL:
Initiative type: ---
Sponsorship status: ---
Comma delimited list of Sponsors:
Crowdfunding goal: 0
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
26.05.00
Circulation function:


Attachments
Bug 41163: Do not decode branchcode in circ log (1.37 KB, patch)
2025-10-31 18:03 UTC, Andrew Fuerste-Henry
Details | Diff | Splinter Review
Bug 41163: Do not decode branchcode in circ log (1.42 KB, patch)
2025-12-13 15:31 UTC, Roman Dolny
Details | Diff | Splinter Review
Bug 41163: Do not decode branchcode in circ log (1.48 KB, patch)
2026-01-23 15:59 UTC, Lisette Scheer
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Fuerste-Henry 2025-10-31 17:53:22 UTC
To recreate:
- confirm circ logging is on
- check out an item at Centerville (CPL)
- go to the log viewer, find your checkout, confirm the Info column includes "CPL"
- switch your logged-in library to Fairfield (FFL)
- confirm the log viewer now shows "FFL" in the info column for your checkout
- use a report to view action_logs.info for your checkout, confirm the field contains "CPL"
Comment 1 Andrew Fuerste-Henry 2025-10-31 18:03:14 UTC
Created attachment 188829 [details] [review]
Bug 41163: Do not decode branchcode in circ log

To test:
1 - confirm circ logging is on
2 - check out an item at Centerville (CPL)
3 - go to the log viewer, find your checkout, confirm the Info column includes "CPL"
4 - switch your logged-in library to Fairfield (FFL)
5 - confirm the log viewer now shows "FFL" in the info column for your checkout
6 - use a report to view action_logs.info for your checkout, confirm the field contains "CPL"
7 - apply patch, restart_all
8 - repeat steps 3-5, confirm "info" column shows "CPL" regardless of logged-in branch
Comment 2 Roman Dolny 2025-12-13 15:31:00 UTC
Created attachment 190481 [details] [review]
Bug 41163: Do not decode branchcode in circ log

To test:
1 - confirm circ logging is on
2 - check out an item at Centerville (CPL)
3 - go to the log viewer, find your checkout, confirm the Info column includes "CPL"
4 - switch your logged-in library to Fairfield (FFL)
5 - confirm the log viewer now shows "FFL" in the info column for your checkout
6 - use a report to view action_logs.info for your checkout, confirm the field contains "CPL"
7 - apply patch, restart_all
8 - repeat steps 3-5, confirm "info" column shows "CPL" regardless of logged-in branch

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Comment 3 Andrew Fuerste-Henry 2026-01-23 14:51:58 UTC
*** Bug 41590 has been marked as a duplicate of this bug. ***
Comment 4 Lisette Scheer 2026-01-23 15:59:24 UTC
Works great, thanks!
Comment 5 Lisette Scheer 2026-01-23 15:59:28 UTC
Created attachment 191961 [details] [review]
Bug 41163: Do not decode branchcode in circ log

To test:
1 - confirm circ logging is on
2 - check out an item at Centerville (CPL)
3 - go to the log viewer, find your checkout, confirm the Info column includes "CPL"
4 - switch your logged-in library to Fairfield (FFL)
5 - confirm the log viewer now shows "FFL" in the info column for your checkout
6 - use a report to view action_logs.info for your checkout, confirm the field contains "CPL"
7 - apply patch, restart_all
8 - repeat steps 3-5, confirm "info" column shows "CPL" regardless of logged-in branch

Signed-off-by: Roman Dolny <roman.dolny@jezuici.pl>
Signed-off-by: Lisette Scheer <lisette@bywatersolutions.com>
Comment 6 Lucas Gass (lukeg) 2026-01-28 21:26:22 UTC
Nice work everyone!

Pushed to main for 26.05