Page MenuHomePhabricator

Link user name in People log view to user page
ClosedPublic

Authored by aklapper on Sep 28 2016, 2:37 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Feb 21, 12:33 PM
Unknown Object (File)
Wed, Feb 19, 3:23 PM
Unknown Object (File)
Thu, Feb 13, 8:15 PM
Unknown Object (File)
Fri, Feb 7, 4:32 AM
Unknown Object (File)
Feb 1 2025, 3:45 PM
Unknown Object (File)
Jan 27 2025, 12:00 AM
Unknown Object (File)
Jan 24 2025, 5:34 AM
Unknown Object (File)
Jan 22 2025, 7:25 AM
Subscribers

Details

Summary

Looking at IPs who recently registered more than one account in
Phabricator and trying to figure out whether they are spam bots
or just all on the same university network, I often want to check
recent user activity of these accounts. Hence linking the entries
in the User column to their user page comes in handy.

Test Plan

Tested on local instance and works as expected.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 13899
Build 17990: arc lint + arc unit

Event Timeline

aklapper retitled this revision from to Link user name in People log view to user page.
aklapper updated this object.
aklapper edited the test plan for this revision. (Show Details)

Possibly simpler fix inline?

src/applications/people/view/PhabricatorUserLogView.php
75

If you just change this line like this, does that also work?

- $handles[$log->getUserPHID()]->getName(),
+ $handles[$log->getUserPHID()]->renderLink(),
aklapper edited edge metadata.
  • Link user name in people log view to user page
This revision is now accepted and ready to land.Sep 28 2016, 3:47 PM
This revision was automatically updated to reflect the committed changes.

Indeed, that one-liner works (nice API).
Thanks for the extremely fast feedback, as usual!