bug 1980 [3/3]: remove bug workaround from pagination links
authorAndrew Moore <andrew.moore@liblime.com>
Thu, 10 Jul 2008 14:12:38 +0000 (09:12 -0500)
committerJoshua Ferraro <jmf@liblime.com>
Thu, 10 Jul 2008 14:18:30 +0000 (09:18 -0500)
yesterday, the C4::Output::pagination_bar had a bug in it that I was crudely working around.
atz fixed that bug this morning. This patch removes the workaround. Thanks, atz!

I'm sending along atz's patch again with my signoff. It needs to be applied before this, as does the
other 1980 patch. I have sent that along (again), too, with a modified subject line. That makes a total of 3 patches for 1980.

Signed-off-by: Joshua Ferraro <jmf@liblime.com>
acqui/neworderbiblio.pl
cataloguing/addbooks.pl

index d1f1932..c314ea9 100755 (executable)
@@ -127,8 +127,7 @@ $template->param(
     opaclayoutstylesheet => C4::Context->preference("opaclayoutstylesheet"),
     opaccolorstylesheet  => C4::Context->preference("opaccolorstylesheet"),
     "BiblioDefaultView".C4::Context->preference("IntranetBiblioDefaultView") => 1,
-    # FIXME: pagination_bar doesn't work right with only one pair of CGI params, so I put two in.
-    pagination_bar       => pagination_bar( "$ENV{'SCRIPT_NAME'}?bug=fix&q=$query&", getnbpages( $total_hits, $results_per_page ), $page, 'page' ),
+    pagination_bar       => pagination_bar( "$ENV{'SCRIPT_NAME'}?q=$query&", getnbpages( $total_hits, $results_per_page ), $page, 'page' ),
 );
 
 # BUILD THE TEMPLATE
index 8e5327e..2f43feb 100755 (executable)
@@ -84,8 +84,7 @@ if ($query) {
         total          => $total_hits,
         query          => $query,
         resultsloop    => \@newresults,
-        # FIXME: pagination_bar doesn't work right with only one pair of CGI params, so I put two in.
-        pagination_bar => pagination_bar( "/cgi-bin/koha/cataloguing/addbooks.pl?bug=fix&q=$query&", getnbpages( $total_hits, $results_per_page ), $page, 'page' ),
+        pagination_bar => pagination_bar( "/cgi-bin/koha/cataloguing/addbooks.pl?q=$query&", getnbpages( $total_hits, $results_per_page ), $page, 'page' ),
     );
 }