]> git.evergreen-ils.org Git - working/Evergreen.git/commit
LP#1795906: Queue position parity for OPAC and Record -> View Holds UIs
authorMike Rylander <mrylander@gmail.com>
Wed, 3 Oct 2018 14:05:09 +0000 (10:05 -0400)
committerGalen Charlton <gmc@equinoxinitiative.org>
Fri, 5 Oct 2018 19:03:26 +0000 (15:03 -0400)
commita242104a70179c3980a929bdaa1f9a4a4952efd5
tree7d7948351bf9673cd9fc0fd7e8fc9983a13d8ed3
parentbbe067af8deaba58c353dd3a2b82e09c5d6c077c
LP#1795906: Queue position parity for OPAC and Record -> View Holds UIs

With the performance driven changes to Hold Shelf and Record -> View Holds
UIs, and backing APIs, there is a discrepancy between conception of queue
position in those UIs and that of others. A simple change will bring other
UIs in line with the new calculation without sacrificing performance in any
of them. In fact, other UIs will be (slightly) faster in most cases.

This change makes all queue position calculations relative to the context
hold's pickup library. This retains the order relative to other directly
relevant holds, though does not show the global position. The initial driver
of this request is parity between the My OPAC queue position, which can
optionally be displayed to patrons, and the Record -> View Holds staff UI.

NOTE: All queue position values are FIFO-based estimates and do not
perfectly model reality, especially in a resource sharing consortium and
even for instances using FIFO hold capture across the board, due to transits,
exceptions, and human staff interaction with the hold queue.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Signed-off-by: Galen Charlton <gmc@equinoxinitiative.org>
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm