Reporter: needs a link Item -> Combined Aged and Active Circulations

[[e-mail]]

Sometimes we need to report on every title in a selection collection with some circ data, which may be aged already.

[Chris Sharp (chrissharp123)]

A PINES library just hit the need for this issue as well. I think adding links between the Combined Aged and Active Circulations and the Aged (patronless) Circulation source, would be appropriate. If others see even more linkage that would be appropriate, please comment here.

I'll take a crack at getting this developed.

[Chris Sharp (chrissharp123)]

My proposed solution:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/csharp/reporter_link_item_to_all_circulations

[Mike Rylander (mrylander)]

Chris,

I have one request that impacts consistency and maintainability. Please pluralize the field name (and, as a consequence, the link name) to match the existing "circulations" field. Because IDL link fields are available in the code, and not just behind the scenes in the reporter, we need to follow the pluralization conventions everywhere.

Otherwise, looks good.

Thanks!

[Chris Sharp (chrissharp123)]

MIke,

Working branch updated with pluralization.

Thanks for your review!

Chris

[Ben Shum (bshum)]

Pushed to master for inclusion in 2.7.0. Thanks Chris for the patch and Mike for the quick review.