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.
*** Bug 10623 has been marked as a duplicate of this bug. ***
This is a regression caused by Bug 11703
Created attachment 33526 [details] [review] 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
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.
(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.
Created attachment 33566 [details] [review] 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
Created attachment 33567 [details] [review] [SIGNED-OFF] 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.
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.
Patch pushed to master. Thanks Kyle!
Pushed to 3.16.x, will be in 3.16.7