Bug 38117

Summary: "Item was not checked in" should not always show
Product: Koha Reporter: Nick Clemens (kidclamp) <nick>
Component: CirculationAssignee: Nick Clemens (kidclamp) <nick>
Status: Pushed to stable --- QA Contact: Martin Renvoize (ashimema) <martin.renvoize>
Severity: normal    
Priority: P5 - low CC: gmcharlt, kyle.m.hall, lucas, martin.renvoize, mspinney, rcoert, sally.healey
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.11.00,24.05.06
Circulation function:
Attachments: Bug 38117: Only show 'not checked in' message if item is checked out
Bug 38117: Only show 'not checked in' message if item is checked out
Bug 38117: Only show 'not checked in' message if item is checked out

Description Nick Clemens (kidclamp) 2024-10-08 11:11:44 UTC
From bug 8461 we added the "Item was not checked in" message marked as 'problem' to let users know when item returns were blocked or failed.

The message is now showing as well for items that were not checked out, and the problem styling makes staff think there is an issue.

We should not show the 'not checked in' message unless the item has a due date/is checked out
Comment 1 Nick Clemens (kidclamp) 2024-10-08 11:16:07 UTC
Created attachment 172517 [details] [review]
Bug 38117: Only show 'not checked in' message if item is checked out

To test:
 1 - Enable system preference 'ShowAllCheckins'
 2 - Set system preference 'BlockReturnOfWithdrawnItems' to 'Block'
 3 - Check in an item
 4 - Note 'Item was not checked in message'
 5 - Confirm it happens for any item that is not currently issued
 6 - Check out an item to a patron
 7 - Mark the item as withdrawn
 8 - Check the item in
 9 - Note message 'Item was not checked in' appears too
10 - Apply patch
11 - Confirm checking in an item not checked out only shows 'Not checked out'
12 - Confirm the withdrawn item still shows 'Item was not checked in'
13 - Confirm above message remains whne checking in more items that were not issued
Comment 2 PTFS Europe Sandboxes 2024-10-08 12:31:08 UTC
Created attachment 172519 [details] [review]
Bug 38117: Only show 'not checked in' message if item is checked out

To test:
 1 - Enable system preference 'ShowAllCheckins'
 2 - Set system preference 'BlockReturnOfWithdrawnItems' to 'Block'
 3 - Check in an item
 4 - Note 'Item was not checked in message'
 5 - Confirm it happens for any item that is not currently issued
 6 - Check out an item to a patron
 7 - Mark the item as withdrawn
 8 - Check the item in
 9 - Note message 'Item was not checked in' appears too
10 - Apply patch
11 - Confirm checking in an item not checked out only shows 'Not checked out'
12 - Confirm the withdrawn item still shows 'Item was not checked in'
13 - Confirm above message remains whne checking in more items that were not issued

Signed-off-by: Michelle Spinney <mspinney@clamsnet.org>
Comment 3 Martin Renvoize (ashimema) 2024-10-24 06:46:23 UTC
Created attachment 173251 [details] [review]
Bug 38117: Only show 'not checked in' message if item is checked out

To test:
 1 - Enable system preference 'ShowAllCheckins'
 2 - Set system preference 'BlockReturnOfWithdrawnItems' to 'Block'
 3 - Check in an item
 4 - Note 'Item was not checked in message'
 5 - Confirm it happens for any item that is not currently issued
 6 - Check out an item to a patron
 7 - Mark the item as withdrawn
 8 - Check the item in
 9 - Note message 'Item was not checked in' appears too
10 - Apply patch
11 - Confirm checking in an item not checked out only shows 'Not checked out'
12 - Confirm the withdrawn item still shows 'Item was not checked in'
13 - Confirm above message remains whne checking in more items that were not issued

Signed-off-by: Michelle Spinney <mspinney@clamsnet.org>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Katrin Fischer 2024-10-24 11:39:06 UTC
Pushed for 24.11!

Well done everyone, thank you!
Comment 5 Lucas Gass (lukeg) 2024-11-20 18:45:19 UTC
Backported to 24.05.x for upcoming 24.05.06