Comment 5 for bug 1695007

Revision history for this message
Bill Erickson (berick) wrote :

Branch pushed (using "aacs").

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1695007-all-circs-slim

Tip commit is a minor bug fix to how circulation summary data is accessed in the browser client copy status. I kept it separate in case we want to merge it first/separately.

From the 2nd commit:

Adds a new view action.all_circulation_slim which collects all action.circulation and action.aged_circulation rows into a single set, without the added joins for copy and patron data imposed by the existing action.aged_circulation view.

The new leaner view is now used by various other views and APIs in place of the all_circulation view:

DB: action.all_circ_chain()
DB: action.summarize_all_circ_chain()
DB: rating.percent_time_circulating()
API: open-ils.circ.retrieve
API: open-ils.circ.copy_checkout_history.retrieve
API: open-ils.circ.copy_details.retrieve
API: open-ils.circ.renewal_chain.retrieve_by_circ[.summary]
API: open-ils.circ.prev_renewal_chain.retrieve_by_circ[.summary]

To Test (webstaff style):

[1] Open a copy in the web staff Item Status / Detail View interface that has live and aged circulations.
[2] Open the Recent Circ History tab
[3] Confirm relevant fields for both types of circs display as expected and that the Patron field for the aged circulation shows a value of <Aged Circulation>