Page MenuHomePhabricator

Pages should not need to be re-marked for translation if page text is the same as the last translated version
Open, LowPublic

Description

Currently, the following happens:

  1. Page is marked for translation
  2. Vandalism is added to page
  3. Vandalism is reverted
  4. Page still shows up in Special:PageTranslation as having pending changes and must be marked for translation again.

When the combined "changes" diff is empty, it is a waste of time to mark the page for translation again. At very least pages with empty "changes" diffs should be hidden from Special:PageTranslation.

Event Timeline

I’m not sure this change would be safe to do. Consider the following scenario:

  1. Page is marked for translation.
  2. Someone makes a constructive edit to it.
  3. A vandal reverts this edit.

This looks the same from the extension’s perspective, however, in this case the page should appear on Special:PageTranslation.

I do not see anything unsafe in this. If the content is exactly the same (including name of the page!) then there isn't anything but one line of metadata to update. I don't this tool is to help with vandalism monitoring, that is the job for other tools.

In fact, we already do this for things like protection which create a "null" revision. We don't ask to re-mark the again just for that.