Bug 21608: Disable dropdown for found holds - add button to revert
authorNick Clemens <nick@bywatersolutions.com>
Thu, 25 Oct 2018 11:45:25 +0000 (11:45 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Tue, 11 Dec 2018 19:13:30 +0000 (19:13 +0000)
commit3c50245cfe272dc105033ff2e59f8e2689f7dd26
tree86f71cfd8f8aa850eefab622e5f1ce92adb410ad
parente5585a10d5f2ee24cad94b2cf17df1ce13e8a01a
Bug 21608: Disable dropdown for found holds - add button to revert

This patch disables the dropdown for found holds, and adds a new button
to revert the waiting status, setting the hold to priority 1

Additionally we remove some changes from 19469 and update the JS to skip
found holds when updating priority

To test:
1 - Find a record with multiple items
2 - Place 4 holds (or more)
3 - Capture one ohld as waiting, one as in transit
4 - View the holds on the record - switch the last to priority one
5 - Waiting and transit statuses get confused
6 - Apply patch
7 - Observe dropdown is now disabled for waiting holds
8 - Confirm other holds operate as expected
9 - Confirm 'Revert found status' resets hold

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Bug 21608: (follow-up) Use RevertWaitingStatus and do not alter _FixPriority

[EDIT]
Completely removed the changes to Reserves.pm by adding module prefix in
the request.pl script.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/includes/holds_table.inc
koha-tmpl/intranet-tmpl/prog/en/modules/reserve/request.tt
reserve/request.pl