[Koha-bugs] [Bug 35431] TransportCostMatrix should be checked after HoldsQueuePrioritizeBranch

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=35431

Nick Clemens <nick at bywatersolutions.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |INVALID
           Assignee|koha-bugs at lists.koha-commun |nick at bywatersolutions.com
                   |ity.org                     |
             Status|Needs Signoff               |RESOLVED

--- Comment #3 from Nick Clemens <nick at bywatersolutions.com> ---
This was a lesson in how the holds queue works, the TCM is skipped here in the
case where there are items available/held at the pickup location of the hold,
even when checked in this piece of the code, we are looking for
HoldsQueuePrioritizeBranch matches

We set the 'holdingbranch' variable here to either the pickup library (if there
are items held there) or to the lowest cost branch.

The code next checks if there are items where the items home branch and the
holds pickup location match, and fills those - trying to get items back to
where they are from

If there are none, then the holdingbranch variable is checked to see if items
from that library can fill - this is the actual matrix checking

Filing a new bug for adding more comments to the routine here

-- 
You are receiving this mail because:
You are watching all bug changes.
You are the assignee for the bug.