Page MenuHomePhabricator

replace.py should report number of changed pages
Closed, ResolvedPublic

Description

compat version had at the end of run message

N pages were changed

in core is missing this info


Version: core-(2.0)
Severity: normal

Details

Reference
bz71828

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 3:56 AM
bzimport set Reference to bz71828.
bzimport added a subscriber: Unknown Object (????).

Change 165654 had a related patch set uploaded by Mpaa:
replace.py: report number of changed pages

https://gerrit.wikimedia.org/r/165654

I wrote this function to trunk but it was not perfect as didn't watch the threads. Just made a simple count of initiated page saves without checking the actual saves, which is good enough in most cases (a page deletion/protection between loading and saving or an edit conflict is not too likely). I have never used and developed rewrite. Once this function is ported to core, should be written better than the original.

Change 165654 merged by jenkins-bot:
replace.py: report number of changed pages

https://gerrit.wikimedia.org/r/165654