Bug 22021: Improve item status display when placing holds in staff
commit89685c15a58c7697f93c8c99f6617168b29a8ee5
authorKatrin Fischer <katrin.fischer.83@web.de>
Sat, 13 Jul 2019 12:23:19 +0000 (13 14:23 +0200)
committerLucas Gass <lucas@bywatersolutions.com>
Tue, 13 Aug 2019 20:23:26 +0000 (13 20:23 +0000)
treedc02eccc64fd3f993061ac06d440ad8698e00adc
parent44a011c3e0f680a75c1b338c62c9fce068dc2919
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>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
(cherry picked from commit d7e6ae38e431c4d6397c057b0696e03f5a67ce14)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
(cherry picked from commit 7468abdf951cfd70a70ff334ecbcde5187536518)

Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
reserve/request.pl