From 775c74f5663bd900e03063fd37d39add5f626d57 Mon Sep 17 00:00:00 2001
From: Joe Atzberger <joe.atzberger@liblime.com>
Date: Mon, 19 May 2008 15:36:37 -0500
Subject: [PATCH] BUGFIX - pagination bar was not in template, so only the
 first 20 hits were

 pagination bar was not in template, so only the first 20 hits were
 visible.  Probably it wasn't included because the script referenced a bogus base URL.
 Also debugified warn statement in script.  Note FIXME for bogus template reference.
 We probably don't need that template or the related selector lines for it.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
---
 .../en/modules/labels/pcard-members-search.tmpl     |  1 +
 labels/pcard-member-search.pl                       | 13 +++++++------
 2 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/labels/pcard-members-search.tmpl b/koha-tmpl/intranet-tmpl/prog/en/modules/labels/pcard-members-search.tmpl
index 4d3dc9a3ab..2c6bc6d287 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/modules/labels/pcard-members-search.tmpl
+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/labels/pcard-members-search.tmpl
@@ -84,6 +84,7 @@ function add_item3(borrowernumber){
 
 <!-- TMPL_IF NAME="resultsloop" -->
 <div id="searchheader"> <h3>Results <!-- TMPL_VAR Name ="from" --> to <!-- TMPL_VAR Name ="to" --> of <!-- TMPL_VAR Name ="numresults" --> found for '<span class="ex"><!-- TMPL_VAR NAME="member" --></span>'</h3></div>
+<!-- TMPL_IF NAME="paginationbar" --><div id="pagination_top"><!-- TMPL_VAR Name ="paginationbar" --></div><!-- /TMPL_IF -->
 <form name="resultform" action="/cgi-bin/koha/labels/pcard-member-search.pl" method="get" class="checkboxed"><div style="float: right; margin-top: .5em;"><input type="submit" class="icon addchecked" value="Add checked" onclick="add_item('checked',<!-- TMPL_VAR NAME="batch_id" -->,'<!-- TMPL_VAR NAME="type" -->'); return false" /> <input type="button" class="close" value="Done" /></div>
 <div style="line-height: 2em; margin-left: .7em;"><a id="CheckAll" href="/cgi-bin/koha/labels/pcard-member-search.pl">Select All</a><a id="CheckNone" href="/cgi-bin/koha/labels/pcard-member-search.pl">Clear All</a></div>
 <div class="searchresults">
diff --git a/labels/pcard-member-search.pl b/labels/pcard-member-search.pl
index d40da1b659..484ff31506 100755
--- a/labels/pcard-member-search.pl
+++ b/labels/pcard-member-search.pl
@@ -27,13 +27,14 @@ use C4::Auth;
 use C4::Output;
 use CGI;
 use C4::Members;
+use C4::Debug;
 
 
 my $input = new CGI;
 my $batch_id    = $input->param('batch_id');
 my $batch_type  = $input->param('type');
 
-warn "Passed Batch Id: $batch_id and Type: $batch_type";
+$debug and warn "[In pcard-member-search] Batch Id: $batch_id, and Type: $batch_type";
 
 my $quicksearch = $input->param('quicksearch');
 my $startfrom = $input->param('startfrom')||1;
@@ -42,7 +43,7 @@ my $resultsperpage = $input->param('resultsperpage')||C4::Context->preference("P
 my ($template, $loggedinuser, $cookie);
 if($quicksearch){
     ($template, $loggedinuser, $cookie)
-    = get_template_and_user({template_name => "members/member-quicksearch-results.tmpl",
+    = get_template_and_user({template_name => "members/member-quicksearch-results.tmpl",	# FIXME: template doesn't exist
                  query => $input,
                  type => "intranet",
                  authnotrequired => 0,
@@ -127,14 +128,14 @@ for (my $i=($startfrom-1)*$resultsperpage; $i < $to; $i++){
   push(@resultsdata, \%row);
 }
 my $base_url =
-    'member.pl?&amp;'
+    'pcard-member-search.pl?'
   . join(
     '&amp;',
     map { $_->{term} . '=' . $_->{val} } (
-        { term => 'member', val => $member},
-        { term => 'orderby', val => $orderby },
+        { term => 'member',         val => $member         },
+        { term => 'orderby',        val => $orderby        },
         { term => 'resultsperpage', val => $resultsperpage },
-        { term => 'type',           val => 'intranet' },
+        { term => 'type',           val => 'intranet'      },
     )
   );