Bug 17057: (follow-up) Replace onsubmit attributes
authorOwen Leonard <oleonard@myacpl.org>
Tue, 13 Aug 2019 15:54:30 +0000 (15:54 +0000)
committerMartin Renvoize <martin.renvoize@ptfs-europe.com>
Mon, 19 Aug 2019 09:13:36 +0000 (10:13 +0100)
commit388f5fa0ffef6a1a5720cb1a94f2f9eb7b6dcf8a
tree9e6b16a1e243efc3925522445d9c4621c7201333
parent54d7fe5250eefac32520f520469d5882c0a67628
Bug 17057: (follow-up) Replace onsubmit attributes

This patch removes the "onsubmit" attribute from the place hold form.

To test, apply the patch and test placing a hold:

Begin the process of placing a hold on a single title. On the
place hold screen, uncheck the "Hold next available item" checkbox and
click the "Place hold" button.

You should be get an alert saying "Please select an item to place a
hold." Select an item and click the "Place hold" button again. The hold
should be placed.

This patch affects the form validation for placing multiple holds, but I
don't think the JavaScript ever has a chance to run: The template blocks
the user from placing multiple holds if one of the titles has no items.

Signed-off-by: Maryse Simard <maryse.simard@inlibro.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/reserve/request.tt