Bug 14514 - LocalHoldsPriority and the HoldsQueue conflict with each other
authorKyle M Hall <kyle@bywatersolutions.com>
Thu, 4 Feb 2016 19:41:57 +0000 (19:41 +0000)
committerFrédéric Demians <f.demians@tamil.fr>
Sat, 22 Oct 2016 12:32:21 +0000 (14:32 +0200)
commit7e6365b5323d60adc949f18fe2efa7ac3d67937e
tree79630e6ffd5c17992a91615100bb3a182930d44c
parenta7741759aa501c8545b6acd97c0a2d851a148c12
Bug 14514 - LocalHoldsPriority and the HoldsQueue conflict with each other

It appears that the LocalHoldsPriority feature and the Holds Queue are
fundamentally at odds with each other.

The problem appears to be that both are attempting to choose the best
way to fill holds. When you are using the holds queue and you check in
an item that has been selected by the holds queue builder, that part of
Koha where the LocalHoldsPriority feature lives doesn't get to see all
the holds in order to pick the best one. Instead only the hold selected
by the holds queue builder is returned so to the LocalHoldsPriority
feature, that is only one hold to pick from!

Test Plan:
1) Apply this patch
2) prove t/db_dependent/HoldsQueue.t
3) All tests should pass

Signed-off-by: Barton Chittenden barton@bywatersolutions.com
Signed-off-by: Dani Elder <dani@bywatersolutions.com>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
(cherry picked from commit 43418ae9f088f023b1d398ab30faaebb9c00cbf5)
Signed-off-by: Frédéric Demians <f.demians@tamil.fr>
C4/HoldsQueue.pm
t/db_dependent/HoldsQueue.t