From: Dan Scott <dan@coffeecode.net>
Date: Sat, 10 Mar 2012 19:45:54 +0000 (-0500)
Subject: TPAC: Limit both call numbers and copies in holdings summary
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=f7458373634197e69c5d05dc053ee44cf59c8778;p=working%2FEvergreen.git

TPAC: Limit both call numbers and copies in holdings summary

In search results, we want to limit both the number of copies we get
back (defaulting to a maximum of 5 copies per call number) and the
number of call numbers we get back (defaulting to a maximum of 5 call
numbers), so that we don't get 50 call numbers each with 5 copies back
and wind up grinding Evergreen's performance down to a halt.

Rough thought is that the caller can filter through the (maximum) 25
hits that are returned to present their preferred results. For example,
a site could opt to show 5 different call numbers with just one copy per
call number, or 5 different copies for a single call number, or all 25.

Signed-off-by: Dan Scott <dan@coffeecode.net>
---

diff --git a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Util.pm b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Util.pm
index eba77a3603..66923874dd 100644
--- a/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Util.pm
+++ b/Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Util.pm
@@ -209,7 +209,7 @@ sub get_records_and_facets {
             $unapi_args->{flesh}, 
             $unapi_args->{site}, 
             $unapi_args->{depth}, 
-            'acp=>' . $unapi_args->{flesh_depth}, 
+            'acn=>' . $unapi_args->{flesh_depth} . ',acp=>' . $unapi_args->{flesh_depth}, 
         ]}
     ) for @$rec_ids;