Page MenuHomePhabricator

Use a 4th constant in rc_patrolled for unpatrollable log entries and categorization changes
Open, Needs TriagePublic


In, it has been suggested that rc_patrolled should not be 1 or 2 for unpatrollable log entries (most of them, except for upload entries) and categorization changes. Perhaps, it should be a 4th constant, "PRC_UNPATROLLABLE = 3". We also need to make RCFilters and the API use this 4th constant. I have abandoned my patch for this so that someone else can make a patch.

Event Timeline

Restricted Application added a project: Growth-Team. · View Herald TranscriptFeb 28 2019, 11:24 PM
Restricted Application added a subscriber: Aklapper. · View Herald Transcript

I would appreciate @Catrope's thoughts here but in general, after defining autopatrol state in RC, we had to add lots of bits to the API to handle it properly. See T184791: Make rc_patrolled 3-state instead of 2-state to distinguish autopatrolled edits. It's not as easy as adding just one more state.