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
F13217236: D16620.id40011.diff
Sat, May 18, 4:10 AM
F13211345: D16620.diff
Fri, May 17, 5:41 AM
F13209519: D16620.id40011.diff
Fri, May 17, 12:42 AM
F13195096: D16620.diff
Sun, May 12, 10:07 PM
F13192087: D16620.id.diff
Sun, May 12, 5:03 AM
F13177870: D16620.diff
Wed, May 8, 8:00 PM
Unknown Object (File)
Mon, May 6, 8:42 AM
Unknown Object (File)
Tue, Apr 30, 10:33 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!