LP#1775276: Check In - "Route To" Field Sometimes Incorrect
authorDan Briem <dbriem@wlsmail.org>
Fri, 29 Nov 2019 19:51:06 +0000 (14:51 -0500)
committerJason Stephenson <jason@sigio.com>
Fri, 31 Jan 2020 13:18:09 +0000 (08:18 -0500)
commit851572686b9d3c827bca282449d66299cc929e2e
treec75b4fb2d6654e9bbeadef0ec290db72feb231fa
parentcf5cf1de34846c6449f3c4fb40c2d0a908ec5bfc
LP#1775276: Check In - "Route To" Field Sometimes Incorrect

When open-ils.circ.checkin closes a transit and creates a new one, the
payload only returns the closed transit. The route dialogs perform a
pcrud search to get the most recent transit, but the grid's Route To
field still depends on the transit returned by the API, so received
transits display the old destination in the Route To field instead of
the shelving location or new destination.

This branch returns the most recent route info collected by the route
dialog and, if the most recent transit destination doesn't match the
old one, the new one is assigned to the route_to prop on the
final_resp. It also checks that the transit is open before displaying
the destination in the Route To field so the shelving location
displays if the most recent transit is closed.

To test:
1. Sign in as Library A and check in an item owned by Library B
2. Place a copy hold on the item with a pickup location of Library C
3. Sign in as Library B and check in the item - note the dialog is
   correct but the grid's Route To field displays Library B instead of
   Library C
4. Apply patch
5. Repeat steps 1-3, note the Route To field displays Library C

Signed-off-by: Dan Briem <dbriem@wlsmail.org>
Signed-off-by: John Amundson <jamundson@cwmars.org>
Signed-off-by: Jason Stephenson <jason@sigio.com>
Open-ILS/web/js/ui/default/staff/circ/services/circ.js