Tweaks to 'my lists' paging code user/dbwells/lp1160596_my_list_paging_tweaks
authorDan Wells <dbw2@calvin.edu>
Fri, 13 Sep 2013 16:59:59 +0000 (12:59 -0400)
committerDan Wells <dbw2@calvin.edu>
Fri, 13 Sep 2013 18:42:32 +0000 (14:42 -0400)
commitd32a97681af0bef366b5dc21069c7b3d2d1b940e
tree4e936562651c783f4111db29aff23302e1d823e4
parent33584305c66c9343e64246174cb2b38b53b6db5d
Tweaks to 'my lists' paging code

1) Don't set and refetch the 'my lists' settings if nothing changed.
(minor nit)

2) Get the count through the same code where we get the bookbag data.

For #2, I am concerned with the current duplication of logic. At best
we are doing things twice, and at worst we end up counting a set which
is slightly different than our actual bookbag search results.

In the updated code, I think the count accuracy is subject to
search/superpage limit settings, so lists are effectively limited by
the search settings. If that is a valid concern, we can beef up the
settings for this query.

Signed-off-by: Dan Wells <dbw2@calvin.edu>
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Account.pm