[Koha-patches] [PATCH] Bug 4470 Patron search result pagination bar

[Frédéric Demians]

Obviously, Koha global pagination function could be improved. This patch
do something which belongs to this function. A CPAN module like
Data::Page could help or be an inspiration... 3.4?

There is also room for improvement in they way resultset are
constructed. The whole set is retrieved and just a subset is displayed.
It impacts performances for libraries with a great number of patrons.
The right solution would be:

  - to find the resultset size with SELECT COUNT(*)
  - to retrieved the exact subset with LIMIT and OFFSET SQL arguments
---
 members/member.pl |    3 ++-
 1 files changed, 2 insertions(+), 1 deletions(-)

diff --git a/members/member.pl b/members/member.pl
index 58485b6..e92ab43 100755
--- a/members/member.pl
+++ b/members/member.pl
@@ -147,7 +147,8 @@ $template->param( letters => \@letters );
 
 $template->param(
     paginationbar => pagination_bar(
-        $base_url,  int( $count / $resultsperpage ) + 1,
+        $base_url,
+        int( $count / $resultsperpage ) + ($count % $resultsperpage ? 1 : 0),
         $startfrom, 'startfrom'
     ),
     startfrom => $startfrom,
-- 
1.7.1