Prevent logs from the new special page from showing up in Special:CheckUserLog
Closed, ResolvedPublic2 Estimated Story Points
Actions

Assigned To

Authored By

	Niharika
	Mar 4 2020, 9:59 PM

Description

Goal

For our work on Checkusers-facing logging for Special:Investigate (T245662), we decided to reuse the cu_log table with a special value for cul_type to indicate that the logs are generated by the new special page so we can exclusively show those on the new special page's logs.
Conversely, we want to prevent the logs from the new special page from showing up in the old special page's logs.

Acceptance criteria:

Log records created from the new Special:Investigate page do not show up in Special:CheckUserLog.

Details

	Subject	Repo	Branch	Lines +/-
	Filter Special:Investigate log entries from CheckUser logs	mediawiki/extensions/CheckUser	master	+11 -2

Customize query in gerrit

Related Objects

Mentioned Here: T245662: CU 2.0: Logging for Special:Investigate - Adding the data

Event Timeline

Niharika triaged this task as Medium priority.Mar 4 2020, 9:59 PM

Niharika created this task.

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptMar 4 2020, 9:59 PM

Niharika set the point value for this task to 2.Mar 5 2020, 7:20 PM

Niharika moved this task from Triage/To be Estimated to The Letter Song on the Anti-Harassment board.Mar 5 2020, 8:11 PM

Niharika edited projects, added Anti-Harassment (The Letter Song); removed Anti-Harassment.

Tchanders claimed this task.Mar 11 2020, 6:51 PM

Change 579027 had a related patch set uploaded (by Tchanders; owner: Tchanders):
[mediawiki/extensions/CheckUser@master] Filter Special:Investigate log entries from CheckUser logs

https://gerrit.wikimedia.org/r/579027

gerritbot added a project: Patch-For-Review.Mar 11 2020, 7:16 PM

Tchanders moved this task from Ready 🎬 (ONLY IF YOU HAVE NO MORE CODE TO REVIEW) to Code Review 🔍 on the Anti-Harassment (The Letter Song) board.Mar 11 2020, 7:18 PM

Change 579027 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] Filter Special:Investigate log entries from CheckUser logs

https://gerrit.wikimedia.org/r/579027

Maintenance_bot removed a project: Patch-For-Review.Mar 12 2020, 3:10 PM

dbarratt moved this task from Code Review 🔍 to QA/Testing 🐞 on the Anti-Harassment (The Letter Song) board.Mar 12 2020, 3:16 PM

ReleaseTaggerBot added a project: MW-1.35-notes (1.35.0-wmf.24; 2020-03-17).Mar 12 2020, 4:00 PM

I inserted some rows into the cu_log table with cul_type = investigate. I checked they did not appear in either the Special:CheckUserLog or the API (action=query&list=checkuserlog).

dbarratt closed this task as Resolved.Mar 13 2020, 4:35 PM