Page MenuHomePhabricator

If revisions are revdel'd, articleinfo compares the surrounding edits as if it were one edit
Closed, ResolvedPublic3 Estimated Story Points



Here Diannaa is listed as having the "Max. text added", that's because she revdel'd a series of edits, and xtools compared the last visible diff with the first live diff after it, which is this. She in fact did not add any content, see the revision history. If there's some way to tell that revisions have been revdel'd, we should not take into account the surrounding diffs. I'm guessing the revdel'd edits are not in the revision table, even though it's only the content (and not the author, edit summary, etc) that's been scrubbed from view.

Event Timeline

kaldari set the point value for this task to 3.May 2 2017, 11:43 PM
DannyH triaged this task as Medium priority.May 2 2017, 11:44 PM
DannyH moved this task from Needs Discussion to Up Next on the Community-Tech board.

Done with

The real issue is that rev_len is not being replicated, even though the diff sizes of revdel'd edits are visible in production. See T101631 for more.

As a workaround, all we're doing here is checking if the previous revision has a rev_len of null, and if so, treat target revision as having a diff size of 0. This means if User:Foo made two edits but only the first one was revdel'd, the second still has a diff size of 0 even though it actually made a change. Until T101631 is resolved I don't see a way around this.

kaldari subscribed.

Code looks reasonable. Haven't been able to test since XTool-dev has been down all day :(

Thanks! I think I've fixed xtools-dev, should work now.