Bug 12778

Summary: Regression: Item lost status doesn't show in list of checkouts
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: CirculationAssignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: major    
Priority: P5 - low CC: gmcharlt, kyle.m.hall, mtj, nengard, tomascohen
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Bug Depends on: 11703    
Bug Blocks:    
Attachments: Bug 12778 - Regression: Item lost status doesn't show in list of checkouts
Bug 12778 - Regression: Item lost status doesn't show in list of checkouts
[SIGNED-OFF] Bug 12778 - Regression: Item lost status doesn't show in list of checkouts
[PASSED QA] Bug 12778 - Regression: Item lost status doesn't show in list of checkouts

Description Katrin Fischer 2014-08-17 18:54:47 UTC
When using the longoverdue script it's possible that items marked lost remain on the patron account. I think it's important for staff to see that some items are marked lost - currently the list of checkouts doesn't show any sign of the lost status.
Comment 1 Katrin Fischer 2014-11-02 11:14:54 UTC
*** Bug 10623 has been marked as a duplicate of this bug. ***
Comment 2 Owen Leonard 2014-11-04 13:40:01 UTC
This is a regression caused by Bug 11703
Comment 3 Kyle M Hall 2014-11-13 18:05:43 UTC Comment hidden (obsolete)
Comment 4 Owen Leonard 2014-11-14 02:14:28 UTC
Only the lost status should display. "Available" should not display for items which are not lost.

Checking out 3.14.x and looking at the way it used to look is helpful. I notice that in 3.14.x the damaged status of items also appeared in that column the same way the lost status did.
Comment 5 Kyle M Hall 2014-11-14 12:19:54 UTC
(In reply to Owen Leonard from comment #4)
> Only the lost status should display. "Available" should not display for
> items which are not lost.
> 
> Checking out 3.14.x and looking at the way it used to look is helpful. I
> notice that in 3.14.x the damaged status of items also appeared in that
> column the same way the lost status did.

I didn't take into account the possibility that the description for lost value 0 would be filled in ( it's not in the default data ). I will correct that and see if I can't do the same for the damaged status.
Comment 6 Kyle M Hall 2014-11-14 12:29:10 UTC Comment hidden (obsolete)
Comment 7 Owen Leonard 2014-11-14 13:05:35 UTC Comment hidden (obsolete)
Comment 8 Katrin Fischer 2014-11-15 08:39:08 UTC
Created attachment 33587 [details] [review]
[PASSED QA] Bug 12778 - Regression: Item lost status doesn't show in list of checkouts

When using the longoverdue script it's possible that items marked lost
remain on the patron account. I think it's important for staff to see
that some items are marked lost - currently the list of checkouts
doesn't show any sign of the lost status.

Test Plan:
1) Find a patron with a checked out lost item
2) Note the lost status is not displayed in the checkouts table
3) Apply this patch
4) Refresh the page, note the lost status now displays
5) Repeat this test plan for a damaged item

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Tested successfully with damaged and multiple lost values.

Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de>
Works as described, no problems found.
Comment 9 Tomás Cohen Arazi 2014-11-17 00:22:56 UTC
Patch pushed to master.

Thanks Kyle!
Comment 10 Mason James 2015-01-19 13:28:26 UTC
Pushed to 3.16.x, will be in 3.16.7