LP#1198465 Recombine generate_fines()
authorDan Wells <dbw2@calvin.edu>
Fri, 5 Dec 2014 17:33:09 +0000 (12:33 -0500)
committerDan Wells <dbw2@calvin.edu>
Wed, 21 Jan 2015 22:08:56 +0000 (17:08 -0500)
commit11d73deca0f30103a55a6051921e05ece7011f58
treedb08dbef242d90f15b2924faf100ac94d3f4e6dd
parent161bdad5c10a035441661db9eb55cdaa29399985
LP#1198465 Recombine generate_fines()

The generate_fines() method was split back in 2010.  There is some
concern about the speed of checkin (e.g. via SIP for a sorter, where we
don't care about fines), and this may have been the reason for the
split.  It might also have been the fact that waiting for the storage
xact to finish was causing the circ xact to timeout.

In any case, since we are bringing them into the same xact now, reason
number 2 is invalid, and reason number 1 won't be helped by the split
anymore (since there is no longer any parallel action).  Let's bring it
back together.

Ultimately, as suggested by Mike, we will probably want to refactor a
bit and return a streaming response for sorting location, then a final
response once we have the fine generation figured out.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Circulate.pm