Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page
Open, Needs TriagePublicBUG REPORT
Actions

Assigned To

None

Authored By

	RheingoldRiver
	May 25 2020, 7:37 AM

Description

Steps to Reproduce:

Create a page from an account without editor, in a namespace recognized by Flagged Revs
Use the api to review that one revision from an account with reviewer:

{
    action:"review",
    revid:<revision id>
}

Now the page is marked as reviewed in recent changes, you can look at verify
Move the page to a namespace ignored by Flagged Revs (e.g. User)

Actual Results: The page is now marked as Unreviewed in Recent Changes

Expected Results: The page should still be reviewed in recent changes.

Event Timeline

RheingoldRiver created this task.May 25 2020, 7:37 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMay 25 2020, 7:37 AM

RheingoldRiver renamed this task from Using review API on all revisions of a page doesn't review the page itself to [FlaggedRevs] Using review API on all revisions of a page doesn't review the page itself.May 25 2020, 7:37 AM

RheingoldRiver updated the task description. (Show Details)May 25 2020, 7:44 AM

DannyS712 added a subscriber: DannyS712.May 25 2020, 7:54 AM

Okay so I've found a workaround for my own purposes, which is to leave behind a redirect in the move and then delete the redirect afterwards. This suppresses the "unchecked page." But it's still kind of weird:

Create page - you have an unchecked page at OriginalLocation
Check page - no unchecked page
Move page WITH redirect - now you have a page which exists at OriginalLocation and is not unchecked, but says it's unchecked in Recent Changes
Delete OriginalLocation - no unchecked page anymore

Also, actually this issue seems to exist even if you press the "Review" button on the page, it's not API-specific - I'd thought it was working from pressing the button, but actually it wasn't.

RheingoldRiver renamed this task from [FlaggedRevs] Using review API on all revisions of a page doesn't review the page itself to [FlaggedRevs] Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page.May 25 2020, 8:14 AM

Aklapper renamed this task from [FlaggedRevs] Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page to Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page.May 25 2020, 10:19 AM

Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked pageOpen, Needs TriagePublicBUG REPORTActions

Description

Event Timeline

Moving a reviewed page to a non-FlaggedRevs namespace without redirect leaves behind an unchecked page
Open, Needs TriagePublicBUG REPORT
Actions