Page MenuHomePhabricator

Update default tool links
Closed, ResolvedPublic3 Estimated Story PointsAug 11 2020

Event Timeline

Restricted Application added a subscriber: Aklapper. ยท View Herald TranscriptJun 24 2020, 4:03 PM
ARamirez_WMF changed the subtype of this task from "Task" to "Deadline".

We discussed that links to Special:Contributions and Special:CheckUserLog should always be present, since they tend to be present on Special:CheckUser across different wikis.

The other (external) tools should be defined in the checkuser-investigate-compare-toollinks message.

Niharika triaged this task as Medium priority.Jul 29 2020, 11:20 PM

@Prtksxna I have a few questions:

  • What icons should we put on the contribs and checks links?
  • Should we also put the contribs and checks links on the user cells (as well as the IP cells)?
  • Should we log the use of the contribs and checks links via EventLogging?
  • CheckUserLog is searchable, but InvestigateLog is not, so it makes sense to link to CheckUserLog. However, CheckUserLog doesn't show Special:Investigate searches. Is this a problem?

Change 618322 had a related patch set uploaded (by Tchanders; owner: Tchanders):
[mediawiki/extensions/CheckUser@master] Update tool links in Special:Investigate Compare table

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

@Prtksxna I have a few questions:

  • What icons should we put on the contribs and checks links?

Let's use check for the check links, and userContributions for contribs.

  • Should we also put the contribs and checks links on the user cells (as well as the IP cells)?

Yes.

  • Should we log the use of the contribs and checks links via EventLogging?

If we're logging everything else on that list I think we should log this too. @Niharika, what do you think?

  • CheckUserLog is searchable, but InvestigateLog is not, so it makes sense to link to CheckUserLog. However, CheckUserLog doesn't show Special:Investigate searches. Is this a problem?
In Slack, @Tchanders said:

We could update InvestigateLog to include 'checkuser' and 'investigate' checks, make it searchable, and link to that?

Yes, this makes sense. And I think in the meantime it could just point to InvestigateLog.

In Slack, @Tchanders said:
We could update InvestigateLog to include 'checkuser' and 'investigate' checks, make it searchable, and link to that?

Yes, this makes sense. And I think in the meantime it could just point to InvestigateLog.

I've given the patch a +2, but please create a ticket for that. :)

Here's a task for making Special:InvestigateLog searchable: T259791

Change 618322 merged by jenkins-bot:
[mediawiki/extensions/CheckUser@master] Update tool links in Special:Investigate Compare table

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

dom_walden added a subscriber: dom_walden.

Goes to index.php?title=Special:Contributions&target=<$username/$ip>.

Goes to Special:InvestigateLog (as Thalia mentions in T256280#6365383, you cannot search the logs).

Goes to https://whois.toolforge.org/gateway.py?lookup=true&ip=<$ip>.

Goes to https://ipcheck.toolforge.org/index.php?ip=192.168.121.180.

N.B. I don't have a user with enough edits to be able to access https://ipcheck.toolforge.org, so I have not been able to see it in action. The URL seems correct, though.

For @Prtksxna, here is what it looks like now:
Username:

username_toolinks.png (175ร—257 px, 7 KB)

IP:
ip_toolinks.png (228ร—268 px, 15 KB)

Test environment Local vagrant MediaWiki 1.36.0-alpha (1ccdde7), CheckUser 2.5 (d9a3619).

Thanks for sharing the screenshots @dom_walden. This looks good.