LP#1697954 Hold details API additional fleshing
authorBill Erickson <berickxx@gmail.com>
Wed, 21 Jun 2017 16:09:37 +0000 (12:09 -0400)
committerBill Erickson <berickxx@gmail.com>
Thu, 29 Jun 2017 19:06:17 +0000 (15:06 -0400)
commit86f7ea160cc740e74dcebf0ab293c98cb0d442ee
tree703ddc9a217858c9f795a3b97b5dd93018b5eee7
parent3f19e9f563e5212fb0ecff6298a7f82a884ea1f9
LP#1697954 Hold details API additional fleshing

* Support new flesh options in hold details retrieval API:

include_current_copy
include_usr
include_cancel_cause
include_requestor

* Teach browser client code to use the new flesh options.  This reduces
  the number of API calls significantly for rendering holds grids.

* Add debug logging to existing local-flesh calls to indicate when/if
  additional API fleshing may be needed.

* Remove TODO comment about batching holds to avoid cstore exhaustion,
  which was fixed with LP#1653001.  However, leave the batching in place
  since it noticeably improves UI responsiveness, at the cost of a few
  extra API calls.

Signed-off-by: Bill Erickson <berickxx@gmail.com>
Signed-off-by: Mike Rylander <mrylander@gmail.com>
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm
Open-ILS/web/js/ui/default/staff/circ/services/holds.js