Page MenuHomePhabricator

Add cu_log.cul_reason_id, cu_log.cul_reason_plaintext_id, cu_log.cul_actor and cu_log.cul_period_id to cu_log table
Open, Stalled, Needs TriagePublic

Related Objects

View Standalone Graph
This task is connected to more than 200 other tasks. Only direct parents and subtasks are shown here. Use View Standalone Graph to show more of the graph.

Event Timeline

Will provide a patch and details soon.

You probably need to make a Schema-change ticket and then once merged, one for production

I think the above ticket is a schema-change one. The patch was merged but has been reverted. I presume I will need to have the DB changes merged in then before I request it for production.

Dreamy_Jazz changed the task status from Open to Stalled.Jun 6 2022, 2:29 PM

Not sure that this is the right status, but want to note that I am working on this and it's not ready yet.

Change 803298 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Add cul_reason_id and cul_reason_plaintext_id into cu_log

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

Change 803302 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Add cul_period_id to the cu_log table

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

Dreamy_Jazz renamed this task from Add cu_log.cul_reason_id and cu_log.cul_reason_plaintext_id to Add cu_log.cul_reason_id, cu_log.cul_reason_plaintext_id and cu_log.cul_period to cu_log table.Jun 14 2022, 3:30 PM

I'm currently working on getting the DB changes merged into the extension. Once they are merged this task will likely include some other changes by Zabe to remove duplication of tasks. All would affect the tables created and used exclusively by the CheckUser extension. The extension has just moved to an abstract schema so this may make this process easier.

Change 803298 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Add cul_reason_id and cul_reason_plaintext_id into cu_log

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

Change 814353 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Add cul_period to cu_log

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

Change 814353 abandoned by Dreamy Jazz:

[mediawiki/extensions/CheckUser@master] Add cul_period to cu_log

Reason:

Going with cul_period_id so that any checks made via the API can have their period value logged.

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

Dreamy_Jazz renamed this task from Add cu_log.cul_reason_id, cu_log.cul_reason_plaintext_id and cu_log.cul_period to cu_log table to Add cu_log.cul_reason_id, cu_log.cul_reason_plaintext_id and cu_log.cul_period_id to cu_log table.Thu, Jul 28, 10:06 PM
Dreamy_Jazz renamed this task from Add cu_log.cul_reason_id, cu_log.cul_reason_plaintext_id and cu_log.cul_period_id to cu_log table to Add cu_log.cul_reason_id, cu_log.cul_reason_plaintext_id, cu_log.cul_actor and cu_log.cul_period_id to cu_log table.