Bug 15062: Holds queue with Transport Cost Matrix will transfer item even if transfer...
authorKyle M Hall <kyle@bywatersolutions.com>
Mon, 26 Oct 2015 12:30:57 +0000 (08:30 -0400)
committerTomas Cohen Arazi <tomascohen@theke.io>
Mon, 23 Nov 2015 14:32:53 +0000 (11:32 -0300)
commit2036be0b877103f7bce424f959e2f3c2df30856b
tree70799b8d8d580653ac4c44d781b09e57a2c6dc77
parent5f19b24f86608f11d575b1a8dac8428e93ea4221
Bug 15062: Holds queue with Transport Cost Matrix will transfer item even if transfers disabled

If the transfer of an item from Library A to Library B is disabled in
the Transport Cost Matrix, but the only item on a record is from Library
A, the holds queue will request a transfer of that item from Library A
to Library B even though the rules state such a transfer is not allowed!

Test Plan:
1) Enable the transport cost matrix
2) For simplicity, leave all cells disabled on it, save this state.
   This state means that only items at each location should fill
   holds where the pickup library is that same location.
3) Create a record with one item, set its home and holding branch
   to Library A
4) Create a hold for pickup at Library B
5) Rebuild the holds queue
6) Note the transfer request for the item to Library B
7) Apply this patch
8) Rebuild the holds queue
9) Note the transfer request is gone

Signed-off-by: Andreas Hedström Mace <andreas.hedstrom.mace@sub.su.se>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
C4/HoldsQueue.pm