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>
Tue, 17 Feb 2015 21:04:21 +0000 (16:04 -0500)
commit7f35056bbb7f0760278f4cf187cc457bcadfc09e
tree2732f3504daf0079067079d93ffc790abcb52a02
parent654e10b9afba93e5246947f6075c307fa86b8192
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