Damaged and Lost statuses are showing the generic category of Lost or Damaged instead of the specific status. We've been fixing this issue in other places. This page should also be corrected.
Still valid. More so: if damaged and lost, only damaged shows.
(In reply to Katrin Fischer from comment #1) > Still valid. More so: if damaged and lost, only damaged shows. I do not understand, I see that in the template: 436 [% IF ( itemloo.damaged ) %] 437 <br />Damaged 438 [% END %] I guess we want to displayed the description for the AV DAMAGED. Lost description as well? One above the other?
The category isn't needed as much as the description of the authorized value in that category.
Created attachment 91519 [details] [review] Bug 22021: Improve item status display when placing holds in staff This patch is a bit of a clean-up to bring the item status display more in line with the display on the detail page: - show descrpition of authorised value for not for loan instead of hardcoded text - Show description of authorised value for lost instead of hardcoded text This is also a translatability fix, as the text came from the .pl - Show description of authorised value damaged instead of hardcoded text - Make sequence of status match display on details page: lost - damaged - not for loan To test: - On a record with multiple items - Add different status to the items damaged, lost, not for loan - Make sure you have items with one status and multiple status at the same time - Look at how the status display on the detail page - Place a hold, compare display - Apply patch - Repeat
Created attachment 91536 [details] [review] Bug 22021: Improve item status display when placing holds in staff This patch is a bit of a clean-up to bring the item status display more in line with the display on the detail page: - show descrpition of authorised value for not for loan instead of hardcoded text - Show description of authorised value for lost instead of hardcoded text This is also a translatability fix, as the text came from the .pl - Show description of authorised value damaged instead of hardcoded text - Make sequence of status match display on details page: lost - damaged - not for loan To test: - On a record with multiple items - Add different status to the items damaged, lost, not for loan - Make sure you have items with one status and multiple status at the same time - Look at how the status display on the detail page - Place a hold, compare display - Apply patch - Repeat Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Created attachment 91631 [details] [review] Bug 22021: Improve item status display when placing holds in staff This patch is a bit of a clean-up to bring the item status display more in line with the display on the detail page: - show descrpition of authorised value for not for loan instead of hardcoded text - Show description of authorised value for lost instead of hardcoded text This is also a translatability fix, as the text came from the .pl - Show description of authorised value damaged instead of hardcoded text - Make sequence of status match display on details page: lost - damaged - not for loan To test: - On a record with multiple items - Add different status to the items damaged, lost, not for loan - Make sure you have items with one status and multiple status at the same time - Look at how the status display on the detail page - Place a hold, compare display - Apply patch - Repeat Signed-off-by: Owen Leonard <oleonard@myacpl.org> Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
Nice work! Pushed to master for 19.11.00
Pushed to 19.05.x for 19.05.03
backported to 18.11.x for 18.11.09