Page MenuHomePhabricator

Implement User:Amalthea/culoghelper.js into the CheckUser extension
Closed, ResolvedPublicFeature

Description

Feature summary (what you would like to be able to do and where):
https://en.wikipedia.org/wiki/User:Amalthea/culoghelper.js is a useful tool for the CheckUserLog page that makes the timestamp of a check clickable. When clicked it takes the checkuser to the log page with no filters other than a offset of an hour ahead of the timestamp clicked. The script then scrolls you down to the check that you clicked and highlights it in a light red colour.

Use case(s) (list the steps that you performed to discover that problem, and describe the actual underlying problem which you want to solve. Do not describe only a solution):
This is used on enwiki to find checks that were performed after this check based on the reason being the same.

Benefits (why should this be implemented?):
Moving this to the CheckUser extension would mean that all checkusers on WMF wikis (not just enwiki that have it installed) can use the feature. It would also allow other projects to use this. Moving this to the extension was suggested on checkuser-l.

Extra details
Some adaptation of the code could be done (such as using a class instead of the style HTML attribute for the highlighting), but most of the work will be writing what was already written in PHP instead of JS. The core concepts and some code could be easily copyable over.

Event Timeline

Also, User:Amalthea has been inactive on enwiki since November 2020 so the tool which works fine at the moment may not receive an update if it breaks from changes to the log.

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

[mediawiki/extensions/CheckUser@master] Link the timestamp in the log and add highlights

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

Change 802902 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Add User:Amalthea/culoghelper.js into the extension

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

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

[mediawiki/extensions/CheckUser@master] Minor bug fix for culoghelper.js addition

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

Change 806485 merged by Samtar:

[mediawiki/extensions/CheckUser@master] Minor bug fix for timestamp linking function

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

Dreamy_Jazz moved this task from General / Unsorted to Done on the CheckUser board.

Patch merged and is in the train going out this week.