Fix / improve bookbag list paging user/dbs/lp1003409_paging_fixes
authorDan Scott <dan@coffeecode.net>
Sun, 22 Jul 2012 22:58:11 +0000 (18:58 -0400)
committerDan Scott <dan@coffeecode.net>
Sun, 22 Jul 2012 22:58:11 +0000 (18:58 -0400)
commit261ce76f898a5618212ce0028958a022e77b034f
treece5f22e8c52cd48a545bad73f5d9862771ee0885
parent428e36c9335aec96e2f6d5015bd01ae507825676
Fix / improve bookbag list paging

The prior code used CSS to hide elements, which is a bit weird given
that we can simply avoid creating the element in the first place and
keep the DOM a lot simpler.

Also, the previous code was subject to some bugs if it wasn't given
an explicit offset GET param or paging at the end of a list
(particularly given that the LIMIT parameter ensured that the number of
bookbags returned would always be equal to or less than the LIMIT
parameter).

Finally, the changes related the permanent bookbags didn't pass through
one of the context parameters we needed (bookbag_count) to enable
paging.

Signed-off-by: Dan Scott <dan@coffeecode.net>
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGCatLoader/Account.pm
Open-ILS/src/templates/opac/myopac/lists.tt2