LP#1306666: Be more mindful with transit status unwinding user/miker/lp1306666-abort-transit-item-status
authorMike Rylander <mrylander@gmail.com>
Tue, 16 Aug 2016 16:07:25 +0000 (12:07 -0400)
committerMike Rylander <mrylander@gmail.com>
Tue, 16 Aug 2016 16:07:25 +0000 (12:07 -0400)
commit525beda0cb861574cbdb633fdfda1dd1d7a000eb
treea4f80a2d463406b25fe2e329c33d8932f43b40dc
parent3b14fac34fbcc3f2cf8eff09ffec7434c5ede029
LP#1306666: Be more mindful with transit status unwinding

When unwinding the copy status wrapper provided by transits, we need to
make sure we don't cause problems due to potentially broken legacy data.
One such incarnation is the "hanging transit", or a transit that should
have been closed but, through some series of events was not, and now
risks damaging a circulation that occured after the transit.  To that end,
we will avoid unwinding the copy status based on an open transit unless
the copy is in "In Transit" status.

Secondly, while hold transits aborted by staff are supposed to be followed up
by a scan of the item (either immediately if the item is in hand, or when
the item physically arrives), such does not always happen.  This can lead
to confusion of other staff members.  To help address this, we will leave
these hold transited items in "In Transit" status, so that they will not
appear available at their home location, and not show up on pull lists.

Note that because of this second change these items will not be available for
opportunistic hold capture when they are scanned in the intended, above-
described worflow, as would be the case before this commit.

Signed-off-by: Mike Rylander <mrylander@gmail.com>
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Transit.pm