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)
committerChris Sharp <csharp@georgialibraries.org>
Wed, 19 Aug 2020 15:07:36 +0000 (11:07 -0400)
commitf74fb36070df1ee626e4a6d7053e37be5198b9b5
tree96473bd022f057152216aadc0999ba6f20765d5e
parent388d0097d8f5c2f1feb5cb4fcad0c30ca9afacaf
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