LP#902255: Correct "ILS Event at check in" logic
authorMike Rylander <mrylander@gmail.com>
Thu, 11 Jun 2015 12:49:21 +0000 (08:49 -0400)
committerMike Rylander <mrylander@gmail.com>
Thu, 11 Jun 2015 12:49:21 +0000 (08:49 -0400)
commit793c1e0ece8753867427eb4a3665f3aae7c1ae8c
treedefe7d5ec8f97ae1acaad8b26c5d09f796ea9b38
parent8e0799c2b112802a5744fb5b19af4050d399e37b
LP#902255: Correct "ILS Event at check in" logic

In a classic case of "fix a bug to reveal another" it turn out that
there was a small logic bug hiding in the check in hold-capture code.
The result of this bug was that the wrong ILS Event object was being
returned when we ran into the double-scan hold-capture situation by
overwriting the correct one immediately after we construct it, which
in turn prevented the previous commit from working as designed.

We now return the correct ILS Event immediately so that we can
properly detect the situation.  Two birds with one branch.

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