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)
committerBrendan Gallagher <brendan@bywatersolutions.com>
Tue, 11 Oct 2016 15:44:47 +0000 (15:44 +0000)
commit43418ae9f088f023b1d398ab30faaebb9c00cbf5
tree0068fb64649bdeb8d54c0f0b5a1048b356983f2b
parented63510fdc0e5a8f8681f0c25b442762315c141d
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>
C4/HoldsQueue.pm
t/db_dependent/HoldsQueue.t