Reduce max per page to 100 and make sure pager sets the adjusted entries_per_page

Author: ranguard

lib/MetaCPAN/Web/Role/Request.pm

@@ -26,7 +26,9 @@ sub get_page_size {
     my $default_page_size = shift;
 
     my $page_size = $req->param('size');
-    unless ( is_PositiveInt($page_size) && $page_size <= 500 ) {
+
+    # We no longer support more than 100 results per page
+    unless ( is_PositiveInt($page_size) && $page_size <= 100 ) {
         $page_size = $default_page_size;
     }
     return $page_size;

root/inc/pager.tx

@@ -3,17 +3,17 @@
 <div class="text-center">
   <ul class="pagination">
     <li class="[% if !$pageset.previous_page { 'disabled' } %]">
-      <a href="[% $page_url({ p => $pageset.previous_page }) %]">«</a>
+      <a href="[% $page_url({ p => $pageset.previous_page, size => $pageset.entries_per_page }) %]">«</a>
     </li>
 
     %%  for $pageset.pages_in_set -> $page_num {
     <li [% if $page_num == $pageset.current_page { %] class="active"[% } %]>
-      <a href="[% $page_url({ p => $page_num }) %]">[% $page_num %]</a>
+      <a href="[% $page_url({ p => $page_num, size => $pageset.entries_per_page  }) %]">[% $page_num %]</a>
     </li>
     %%  }
 
     <li class="[% if !$pageset.next_page { 'disabled' } %]">
-      <a href="[% $page_url({ p => $pageset.next_page }) %]">»</a>
+      <a href="[% $page_url({ p => $pageset.next_page, size => $pageset.entries_per_page }) %]">»</a>
     </li>
   </ul>
 </div>