jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/589872 )
Change subject: [bugfix] Remove duplicate counter in replace.py ......................................................................
[bugfix] Remove duplicate counter in replace.py
Page counting is already done BaseBot. Remove duplicate page counter.
Bug: T196851 Change-Id: I57e7e42ebe3d193fcef8481217c5b7b8c045abbb --- M scripts/replace.py 1 file changed, 4 insertions(+), 7 deletions(-)
Approvals: Dvorapa: Looks good to me, approved jenkins-bot: Verified
diff --git a/scripts/replace.py b/scripts/replace.py index ab361b9..7033908 100755 --- a/scripts/replace.py +++ b/scripts/replace.py @@ -578,7 +578,6 @@
self.sleep = sleep self.summary = summary - self.changed_pages = 0 self._pending_processed_titles = Queue()
def isTitleExcepted(self, title, exceptions=None): @@ -667,11 +666,10 @@ new_text = self.apply_replacements(original_text, set(), page=page) return new_text
- def _count_changes(self, page, err): - """Count successfully changed pages; log changed titles for display.""" + def _log_changes(self, page, err): + """Log changed titles for display.""" # This is an async put callback if not isinstance(err, Exception): - self.changed_pages += 1 self._pending_processed_titles.put((page.title( as_link=True), True)) else: # unsuccessful pages @@ -680,11 +678,11 @@
def _replace_async_callback(self, page, err): """Callback for asynchronous page edit.""" - self._count_changes(page, err) + self._log_changes(page, err)
def _replace_sync_callback(self, page, err): """Callback for synchronous page edit.""" - self._count_changes(page, err) + self._log_changes(page, err) if isinstance(err, Exception): raise err
@@ -1200,7 +1198,6 @@ # Explicitly call pywikibot.stopme(). It will make sure the callback is # triggered before replace.py is unloaded. pywikibot.stopme() - pywikibot.output('\n{0} pages changed.'.format(bot.changed_pages))
if __name__ == '__main__':
pywikibot-commits@lists.wikimedia.org