Changeset 98 for trunk/dns.cgi

Timestamp:

06/24/11 17:30:23 (14 years ago)

Author:

Kris Deugau

Message:

/trunk

Fiddle results-per-page default
Bugfix for offset on filtering a result set close enough to

the end of the original full set that the filtered set
doesn't have enough results to reach that offset.

File:

• trunk/dns.cgi (modified) (2 diffs)

trunk/dns.cgi

@@ -101 +101 @@
 # default
 #my $perpage = 15;
-my $perpage = 3;
+my $perpage = 5;
 my $offset = ($webvar{offset} ? $webvar{offset} : 0);
 
@@ -1348 +1348 @@
   my $parent = shift;
 
+  # Fix display/UI bug where if you are not on the first page of the list, and
+  # you add a search term or click one of the "starts with" links, you end up
+  # on a page showing nothing.
+  # For bonus points, this reverts to the original offset on clicking the "All" link (mostly)
+  $offset-- while ($offset * $perpage) >= $pgcount;
+
   $page->param(ntot => $pgcount);
   $page->param(nfirst => (($offset eq 'all' ? 0 : $offset)*$perpage+1));