From: MusikAnimal <musikanimal@gmail.com>
Date: Tue, 19 Sep 2017 23:51:34 +0000 (-0400)
Subject: Yet another attempt to fix the populateIpChanges script
X-Git-Tag: 1.31.0-rc.0~2033
X-Git-Url: https://git.heureux-cyclage.org/?p=lhc%2Fweb%2Fwiklou.git;a=commitdiff_plain;h=ffba23d177d5eee876bdc59cdf1b945281b18041

Yet another attempt to fix the populateIpChanges script

It would seem we'd need to update $blockStart if there are no results
for a given block, and more importantly, continue and not break!

Bug: T175962
Change-Id: Ice1bdae3d16cf365da14c6df0e8d91d2b954e064
---

diff --git a/maintenance/populateIpChanges.php b/maintenance/populateIpChanges.php
index 17f49ee358..f7bcc12c3d 100644
--- a/maintenance/populateIpChanges.php
+++ b/maintenance/populateIpChanges.php
@@ -88,7 +88,8 @@ TEXT
 			$numRows = $rows->numRows();
 
 			if ( !$rows || $numRows === 0 ) {
-				break;
+				$blockStart = $blockEnd + 1;
+				continue;
 			}
 
 			$this->output( "...checking $numRows revisions for IP edits that need copying, " .