LP#1697954 Hold details API additional fleshing user/berick/lp1697954-signoff-tweaks
authorBill Erickson <berickxx@gmail.com>
Wed, 21 Jun 2017 16:09:37 +0000 (12:09 -0400)
committerBill Erickson <berickxx@gmail.com>
Wed, 21 Jun 2017 16:09:39 +0000 (12:09 -0400)
commit6577164af2ba67067d152a30462be94f1d2466a0
treec825a3777a5054e6f96494a8aabfa951d97d6f86
parentc5bf3ff4509d1f3107bf74544b026f68a5d41c05
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>
Open-ILS/src/perlmods/lib/OpenILS/Application/Circ/Holds.pm
Open-ILS/web/js/ui/default/staff/circ/services/holds.js