action.item_user_circ_test reverted back to old version with 3.10 upgrade
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Medium
|
Unassigned |
Bug Description
The action.
-- Fail if the item isn't in a circulateable status on a non-renewal
IF NOT renewal AND item_object.status <> 8 AND item_object.status NOT IN (
(SELECT id FROM config.copy_status WHERE is_available) ) THEN
done := TRUE;
RETURN NEXT result;
-- Alternately, fail if the item isn't checked out on a renewal
It looks like the current upgrade scripts to 3.10 reverts that change and goes back to hardcoded values, which is bad,
-- Fail if the item isn't in a circulateable status on a non-renewal
IF NOT renewal AND item_object.status NOT IN ( 0, 7, 8 ) THEN
done := TRUE;
RETURN NEXT result;
I'm not sure when this old code got put back in there, but it should be corrected.
Changed in evergreen: | |
assignee: | nobody → Jeff Davis (jdavis-sitka) |
tags: | added: regression |
Changed in evergreen: | |
status: | New → Confirmed |
importance: | Undecided → Medium |
Changed in evergreen: | |
assignee: | nobody → Galen Charlton (gmc) |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
It looks like it happened during LP#1411819