Currently in 24.05 and in master, after setting an item to damaged, this status does not appear in the detail.pl page. The damaged status does show if the item is checked out, it will appear in the patron's checkouts with the damaged status. It also will show when the item is checked in. If changing the item to LOST or Not for Loan, these values do show. It appears to only not show when applying the item to the damaged status.
Oh, also the marking an item 'withdrawn' from the patron's account, the withdrawn status doesn't show the status in the patron's account but does show the withdrawn status from the bibliographic record page This is the opposite of the first comment on this bug.
Comment 1 would be something else, this is just from bug 33568 using the wrong term to find damaged in the API output it uses to build the holdings table for bib detail.
Created attachment 168003 [details] [review] Bug 37078: Damaged status not showing in detail.pl The API returns the fact that an item is damaged in 'damaged_status' but the code that builds the holdings table is looking for 'damaged' so it doesn't find that an item is damaged. Test plan: 1. Administration - Authorized values - DAMAGED row, click Add 2. Give it the Authorized value 2 and the Description Awful 3. On a record with more than one item, like Empress of the blues, edit one item to have Damaged status: Damaged and one to have Damaged status: Awful 4. On the bib detail view, note that the status column shows both as 'Available' 5. Apply patch, refresh 6. Note that now one shows as Damaged and one as Awful (verifying that the value is being read from the AV, not the fallback default which is also 'Damaged')
Created attachment 168021 [details] [review] Bug 37078: Damaged status not showing in detail.pl The API returns the fact that an item is damaged in 'damaged_status' but the code that builds the holdings table is looking for 'damaged' so it doesn't find that an item is damaged. Test plan: 1. Administration - Authorized values - DAMAGED row, click Add 2. Give it the Authorized value 2 and the Description Awful 3. On a record with more than one item, like Empress of the blues, edit one item to have Damaged status: Damaged and one to have Damaged status: Awful 4. On the bib detail view, note that the status column shows both as 'Available' 5. Apply patch, refresh 6. Note that now one shows as Damaged and one as Awful (verifying that the value is being read from the AV, not the fallback default which is also 'Damaged') Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org>
Simple solution to fairly bad problem, thanks Phil.
Created attachment 168068 [details] [review] Bug 37078: Damaged status not showing in detail.pl The API returns the fact that an item is damaged in 'damaged_status' but the code that builds the holdings table is looking for 'damaged' so it doesn't find that an item is damaged. Test plan: 1. Administration - Authorized values - DAMAGED row, click Add 2. Give it the Authorized value 2 and the Description Awful 3. On a record with more than one item, like Empress of the blues, edit one item to have Damaged status: Damaged and one to have Damaged status: Awful 4. On the bib detail view, note that the status column shows both as 'Available' 5. Apply patch, refresh 6. Note that now one shows as Damaged and one as Awful (verifying that the value is being read from the AV, not the fallback default which is also 'Damaged') Signed-off-by: Brendan Lawlor <blawlor@clamsnet.org> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Pushed for 24.11! Well done everyone, thank you!
Can this be pushed to 24.05? Not showing the correct status is a significant issue for staff who use the holdings information regularly. Thx! --CJ
rather, backported to 24.05 Thx! --CJ
Backported to 24.05.x for upcoming 24.05.02
*** Bug 37372 has been marked as a duplicate of this bug. ***
Depends on Bug 33568 not in 23.11.x