Bug 23434: Hold confirmation dialog problem if HoldsAutoFill is enabled
authorOwen Leonard <oleonard@myacpl.org>
Tue, 6 Aug 2019 13:25:24 +0000 (13:25 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 12 Aug 2019 13:33:52 +0000 (14:33 +0100)
commite1b79db3e52c8041c5fd18fd0ee250bf7636bd4e
tree310d3e4136ed9fa1cb8afe687972766bba3329e2
parent133090902f01d0b5af83091850dda75d13925506
Bug 23434: Hold confirmation dialog problem if HoldsAutoFill is enabled

This patch corrects a problem with hold confirmation dialogs on the
checkin page when the HoldsAutoFill system preference is enabled and the
item being checked in has a hold at another library.

The confirmation dialog is converted to a modal to match other
confirmation information.

As part of this process two whole sections of the template have been
removed because they were redundant.

To test, apply the patch and test the following circumstances both with
HoldsAutoFill enabled AND disabled:

 - A hold for a patron at your library
 - A hold for a patron not at your library
   - With HoldsAutoFill ON, both the "OK" and "Print slip and confirm"
     buttons should dismiss the modal without reloading the page.
 - A hold which is already marked 'Waiting'
 - An item which needs to be transferred
 - An item which has already been transferred

In all cases the confirmation modal should appear with correct
information.

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
koha-tmpl/intranet-tmpl/prog/en/modules/circ/returns.tt