From: Dylan William Hardison <dylan@hardison.net>
Date: Mon, 1 Apr 2019 16:28:13 +0000 (-0400)
Subject: Bug 1506144 follow-up - handle paging more elegantly
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=5f4ae6d136b8708020f7b40067d0e43554bdfc68;p=thirdparty%2Fbugzilla.git

Bug 1506144 follow-up - handle paging more elegantly
---

diff --git a/Bugzilla/App/Command/report_ping.pm b/Bugzilla/App/Command/report_ping.pm
index a08d5f3d6..44543952c 100644
--- a/Bugzilla/App/Command/report_ping.pm
+++ b/Bugzilla/App/Command/report_ping.pm
@@ -23,8 +23,7 @@ sub run {
   my $json
     = JSON::MaybeXS->new(convert_blessed => 1, canonical => 1, pretty => 1);
   my $report_type = 'Simple';
-  my $page        = 1;
-  my ($rows, $base_url, $test, $dump_schema);
+  my ($page, $rows, $base_url, $test, $dump_schema);
 
 
   Bugzilla->usage_mode(USAGE_MODE_CMDLINE);
@@ -57,7 +56,7 @@ sub run {
   if ($test) {
     foreach my $p ($report->page .. $report->pager->last_page) {
       # get the next page, except for page 1.
-      $rs = $rs->page($p) if $p > $report->page;
+      $rs = $rs->page($p);
       say "Testing page $p of ", $report->pager->last_page;
       foreach my $result ($rs->all) {
         my @error = $report->test($result);
@@ -71,7 +70,7 @@ sub run {
   else {
     foreach my $p ($report->page .. $report->pager->last_page) {
       # get the next page, except for page 1.
-      $rs = $rs->page($p) if $p > $page;
+      $rs = $rs->page($p);
       say "Sending page $p of ", $report->pager->last_page;
       Mojo::Promise->all(map { $report->send($_) } $rs->all)->wait;
     }