Bug 22814: Make patron names on holds modal consistent (second check-in)
authorKatrin Fischer <katrin.fischer.83@web.de>
Thu, 10 Oct 2019 12:53:10 +0000 (12:53 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Fri, 1 Nov 2019 08:51:05 +0000 (08:51 +0000)
commit4764c439f9e19e0b1a88a7503e2bb502dbd3b7ac
tree2b0d689ae258628b2785386aa43fa547f89c1de0
parent4fd52650bcb2f8a549ea55d22aadeeac7202d19d
Bug 22814: Make patron names on holds modal consistent (second check-in)

Bug 22203 changed the behavior on the hold-found2 modal
that pops up when a hold is found so that whether or not
the patron's name is displayed is now governed by the
HidePatronName system preference.

However, if a hold is confirmed and you check the item
in a second time, the modal that pops up is hold-found1
and whether or not the patron's name is displayed in
the hold-found1 modal is not governed by the HidePatronName
system preference.

For the sake of consistency, the display of the patron's
name on hold-found2 and hold-found1 should both be governed
by the HidePatronName system preference.

To test:
- Place a hold on an item
- Check in the item to trigger the hold modal
- Confirm hold
- Check the item in again - verify name of patron displays
- Apply patch
- Repeat
- Verify name display now matches between the two modals
- change HidePatronName pref
- Repeat
- Name display should still match

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt