Change Details

## Terminology ## There's a patrol log and patrol/unpatrol actions, which use MediaWiki Core's https://www.mediawiki.org/wiki/Help:Patrolled_edits There's a page curation log and review/unreview actions, which use MediaWiki extension PageTriage https://www.mediawiki.org/wiki/Extension:PageTriage ## The bug ## Assuming that PageTriage is installed... In theory, anytime a page is marked as reviewed, it should also mark the page as patrolled. And anytime a page is marked as unreviewed, it should also mark the page as unpatrolled. However, I have seen plenty of evidence that this does not happen, because "review" actions sometimes do not make it into the "patrol" log. This ticket is to investigate what actions lead to these two logs getting out of sync, and to fix them. ## Things to check ## Note: patrolling is only stored in recentchanges, not anywhere else, so only lasts 30-90 days * [ ] Make sure that marking as reviewed also marks as patrolled * [ ] Make sure that marking as unreviewed also marks as unpatrolled * [ ] Make sure that marking as patrolled also marks as reviewed * ~~Make sure that marking as unpatrolled also marks as unreviewed~~ (unpatrolling is not possible, {T22399}) * [ ] Look up what other actions can cause a review/unreview action, and investigate those * [ ] Page moves * [ ] Deletion * [ ] Undeletion * [ ] Is there a chance that patrolled edits fundamentally works differently than reviewing? For example, maybe patrolled edits is by diff rather than by article, which could cause challenges with keeping them in sync?

## Terminology ## There's a patrol log and patrol/unpatrol actions, which use MediaWiki Core's https://www.mediawiki.org/wiki/Help:Patrolled_edits There's a page curation log and review/unreview actions, which use MediaWiki extension PageTriage https://www.mediawiki.org/wiki/Extension:PageTriage ## The bug ## Assuming that PageTriage is installed... In theory, anytime a page is marked as reviewed, it should also mark the page as patrolled. And anytime a page is marked as unreviewed, it should also mark the page as unpatrolled. However, I have seen plenty of evidence that this does not happen, because "review" actions sometimes do not make it into the "patrol" log. This ticket is to investigate what actions lead to these two logs getting out of sync, and to fix them. ## Things to check ## Note: patrolling is only stored in recentchanges, not anywhere else, so only lasts 30-90 days * [ ] Make sure that marking as reviewed also marks as patrolled * ~~Make sure that marking as unreviewed also marks as unpatrolled~~ * [ ] Make sure that marking as patrolled also marks as reviewed * ~~Make sure that marking as unpatrolled also marks as unreviewed~~ (unpatrolling is not possible, {T22399}) * [ ] Look up what other actions can cause a review/unreview action, and investigate those * [ ] Page moves * [ ] Deletion * [ ] Undeletion * [ ] Is there a chance that patrolled edits fundamentally works differently than reviewing? For example, maybe patrolled edits is by diff rather than by article, which could cause challenges with keeping them in sync?

## Terminology ## There's a patrol log and patrol/unpatrol actions, which use MediaWiki Core's https://www.mediawiki.org/wiki/Help:Patrolled_edits There's a page curation log and review/unreview actions, which use MediaWiki extension PageTriage https://www.mediawiki.org/wiki/Extension:PageTriage ## The bug ## Assuming that PageTriage is installed... In theory, anytime a page is marked as reviewed, it should also mark the page as patrolled. And anytime a page is marked as unreviewed, it should also mark the page as unpatrolled. However, I have seen plenty of evidence that this does not happen, because "review" actions sometimes do not make it into the "patrol" log. This ticket is to investigate what actions lead to these two logs getting out of sync, and to fix them. ## Things to check ## Note: patrolling is only stored in recentchanges, not anywhere else, so only lasts 30-90 days * [ ] Make sure that marking as reviewed also marks as patrolled * [ ] * ~~Make sure that marking as unreviewed also marks as unpatrolled~~ * [ ] Make sure that marking as patrolled also marks as reviewed * ~~Make sure that marking as unpatrolled also marks as unreviewed~~ (unpatrolling is not possible, {T22399}) * [ ] Look up what other actions can cause a review/unreview action, and investigate those * [ ] Page moves * [ ] Deletion * [ ] Undeletion * [ ] Is there a chance that patrolled edits fundamentally works differently than reviewing? For example, maybe patrolled edits is by diff rather than by article, which could cause challenges with keeping them in sync?