OPAC Browse: use superpage concept for performance; fix other counting bug
authorLebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Thu, 23 May 2013 22:23:54 +0000 (18:23 -0400)
committerBen Shum <bshum@biblio.org>
Mon, 15 Jul 2013 15:45:24 +0000 (11:45 -0400)
commite6696aced7206d62eeeaa0674535895bd6a68c46
treedf7f6c8ecbf35d73d2b91eede65b02043366f159
parent5cfed3e1fdc82a8595f8dfaf47d138dac292fcb6
OPAC Browse: use  superpage concept for performance; fix other counting bug

This commit does two totally different things.

1) It implements a superpages concept in the visibility testing that
happens for records linked to displayed browse headings. By default now
we will only test 100 records at at time for visible holdings w/in the
browse scope until at least 1 visible record is found (or the set of
records is exhausted).

In practical terms this will result in display of headings followed by
"at least 100" or "at least 92" instead of larger numbers, but there
should be a big speed win. Yes it's possible, though very unlikely, that
you'd get cases of "at least 1".

When there's less than a browse superpage worth of records to check
anyway, the count should be exact and won't say "at least."

2) It fixes a bug where the counts for the number of bib records linked
to an authority reference heading were not cached properly on the server
side, so upon page reloads or back/forward operations those counts could
disappear.

Signed-off-by: Lebbeous Fogle-Weekley <lebbeous@esilibrary.com>
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Browse.pm
Open-ILS/src/sql/Pg/030.schema.metabib.sql
Open-ILS/src/sql/Pg/950.data.seed-values.sql
Open-ILS/src/sql/Pg/upgrade/YYYY.schema.bib-auth-browse.sql
Open-ILS/src/templates/opac/browse.tt2