Bug 27064: Only allow transferring a hold from the transfers page
authorNick Clemens <nick@bywatersolutions.com>
Fri, 20 Nov 2020 11:34:11 +0000 (11:34 +0000)
committerKyle M Hall <kyle@bywatersolutions.com>
Fri, 18 Jun 2021 11:11:35 +0000 (07:11 -0400)
commit12b1925cd507a3b9522dfeaf4aa8d23a98c46f26
tree9238e5b6c06c936f5293c095d7249c1c998d6e32
parent9d70d1eaec1b265e1e32072ba493cff609350f69
Bug 27064: Only allow transferring a hold from the transfers page

These patches replace the 'Waiting' button on the transfers page with a 'Transfer' button
and correct some other related problem by passing the hold object to the template and
using that to fetch patron info as well as passing the reserve_id through to ensure the correct hold is
affected at all times

To test:
 1 - Place a hold for delivery at Library B
 2 - Sign in at Library A
 3 - Browse to Circulation->Transfers
 4 - Attempt to transfer an item on the title with the hold to Library B
 5 - You get a notice that hold was found (missing patron/branch info)
 6 - You have the option to set the hold waiting - click it
 7 - The transfer is generated and marked completed
 8 - The hold is marked as waiting, but the item is still at Library A and no transfer is active
 9 - The patron is notified that the hold is waiting
10 - Revert the hold or cancel and place a new one
11 - Apply patches
12 - Attempt transfer again
13 - You now have the option to transfer the hold
14 - Click that
15 - Hold is in transit and transfer is generated correctly
16 - Transfer again and choose 'cancel'
17 - Confirm hold is cancelled and transfer generated

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Joonas Kylmälä <joonas.kylmala@helsinki.fi>
Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
circ/branchtransfers.pl
koha-tmpl/intranet-tmpl/prog/en/modules/circ/branchtransfers.tt