The list of issued items on moremember.pl display the lost or damaged status of an item along side the due date, but is otherwise pretty much identical to the one on circulation.pl. For the sake of consistency we should have those statuses show on circulation.pl as well.
Created attachment 13898 [details] [review] Bug 9219 - Show damaged and lost statuses for issues on circulation.pl as they are on moremember.pl This patch adds the lost and damaged status for an issued item to the table of issued items on circulation.pl to make it align more closely to the same table on moremember.pl Test Plan: 1) Apply patch 2) Issue an item 3) Set the item's damaged status 4) Verify the status is showing near the date due on circulation.pl 5) Clear the damaged status and set the lost status again 6) Repeat step 4
This seems to be working well but misses one part of the display: If you have a patron who is a guarantor or a guarantee the page will have a "relative's checkouts" tab which needs the same update.
Created attachment 15255 [details] [review] Bug 9219 - Show damaged and lost statuses for issues on circulation.pl as they are on moremember.pl This patch adds the lost and damaged status for an issued item to the table of issued items on circulation.pl to make it align more closely to the same table on moremember.pl Test Plan: 1) Apply patch 2) Issue an item 3) Set the item's damaged status 4) Verify the status is showing near the date due on circulation.pl 5) Clear the damaged status and set the lost status again 6) Repeat step 4
Created attachment 15256 [details] [review] Bug 9219 - Show damaged and lost statuses for issues on circulation.pl as they are on moremember.pl This patch adds the lost and damaged status for an issued item to the table of issued items on circulation.pl to make it align more closely to the same table on moremember.pl Test Plan: 1) Apply patch 2) Issue an item 3) Set the item's damaged status 4) Verify the status is showing near the date due on circulation.pl 5) Clear the damaged status and set the lost status again 6) Repeat step 4
Created attachment 16124 [details] [review] [SIGNED-OFF] Bug 9219 - Show damaged and lost statuses for issues on circulation.pl as they are on moremember.pl This patch adds the lost and damaged status for an issued item to the table of issued items on circulation.pl to make it align more closely to the same table on moremember.pl Test Plan: 1) Apply patch 2) Issue an item 3) Set the item's damaged status 4) Verify the status is showing near the date due on circulation.pl 5) Clear the damaged status and set the lost status again 6) Repeat step 4 Signed-off-by: Owen Leonard <oleonard@myacpl.org> http://bugs.koha-community.org/show_bug.cgi?id=6739
This sounds like a consistency (or lack thereof) bug to me.
CONFLICT (content): Merge conflict in koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt
Created attachment 17621 [details] [review] Bug 9219 - Show damaged and lost statuses for issues on circulation.pl as they are on moremember.pl This patch adds the lost and damaged status for an issued item to the table of issued items on circulation.pl to make it align more closely to the same table on moremember.pl Test Plan: 1) Apply patch 2) Issue an item 3) Set the item's damaged status 4) Verify the status is showing near the date due on circulation.pl 5) Clear the damaged status and set the lost status again 6) Repeat step 4 Signed-off-by: Owen Leonard <oleonard@myacpl.org> http://bugs.koha-community.org/show_bug.cgi?id=6739
Created attachment 17641 [details] [review] [PASSED QA] Bug 9219 - Show damaged and lost statuses for issues on circulation.pl as they are on moremember.pl This patch adds the lost and damaged status for an issued item to the table of issued items on circulation.pl to make it align more closely to the same table on moremember.pl Test Plan: 1) Apply patch 2) Issue an item 3) Set the item's damaged status 4) Verify the status is showing near the date due on circulation.pl 5) Clear the damaged status and set the lost status again 6) Repeat step 4 Signed-off-by: Owen Leonard <oleonard@myacpl.org> http://bugs.koha-community.org/show_bug.cgi?id=6739 Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> I could only test this with the DAMAGED value, because currently lost items are removed from the patron account when you set the status. I know there is a patch somewhere to make it optional and the change looks alright. Passes all tests and QA script.
This patch has been pushed to master and 3.12.x.