Description
Caroline Cyr La Rose
2023-01-11 18:09:28 UTC
Could this be Bug 32445 - Status display of 'not for loan' items is broken in OPAC/staff ? (In reply to Katrin Fischer from comment #1) > Could this be Bug 32445 - Status display of 'not for loan' items is broken > in OPAC/staff ? That is what I thought too, but I didn't want to presume since the display was already broken before that patch. Created attachment 147260 [details] [review] Bug 32611: Not for loan items don't show the specific not for loan value in OPAC detail page This patch corrects what appear to be some copy-paste errors which resulted in the "not for loan" authorized value description not displaying, instead the generic "Not for loan" is shown. The patch also adds another case for displaying the item restricted status. Previously items.restricted would only display if the item *also* had a notforloan value. To test it's probably easiest to find a record with multiple items and set various item statuses (damaged, notforloan, lost, restricted) and confirm that the right description is shown. Created attachment 147268 [details] [review] Bug 32611: Not for loan items don't show the specific not for loan value in OPAC detail page This patch corrects what appear to be some copy-paste errors which resulted in the "not for loan" authorized value description not displaying, instead the generic "Not for loan" is shown. The patch also adds another case for displaying the item restricted status. Previously items.restricted would only display if the item *also* had a notforloan value. To test it's probably easiest to find a record with multiple items and set various item statuses (damaged, notforloan, lost, restricted) and confirm that the right description is shown. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Created attachment 147400 [details] [review] Bug 32611: Not for loan items don't show the specific not for loan value in OPAC detail page This patch corrects what appear to be some copy-paste errors which resulted in the "not for loan" authorized value description not displaying, instead the generic "Not for loan" is shown. The patch also adds another case for displaying the item restricted status. Previously items.restricted would only display if the item *also* had a notforloan value. To test it's probably easiest to find a record with multiple items and set various item statuses (damaged, notforloan, lost, restricted) and confirm that the right description is shown. Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Pushed to master for 23.05. Nice work everyone, thanks! Nice work everyone! Pushed to stable for 22.11.x Does not apply cleanly to 22.05.x, no backport. Think this has introduced a different regression... |