Bug 22021: Improve item status display when placing holds in staff
authorKatrin Fischer <katrin.fischer.83@web.de>
Sat, 13 Jul 2019 12:23:19 +0000 (14:23 +0200)
committerFridolin Somers <fridolin.somers@biblibre.com>
Tue, 6 Aug 2019 14:36:23 +0000 (16:36 +0200)
commit7468abdf951cfd70a70ff334ecbcde5187536518
tree63a5fd355155e58aa1b567935a7f65827a653002
parent267ad48da77eb3fedd06f1050db0ffc4406477c2
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>
koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
reserve/request.pl