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
F14064546: D16620.diff
Mon, Nov 18, 11:56 PM
F14051325: D16620.id40018.diff
Thu, Nov 14, 11:51 PM
F14009065: D16620.diff
Wed, Oct 30, 9:26 AM
F13984455: D16620.diff
Sun, Oct 20, 1:30 PM
F13977816: D16620.id.diff
Oct 18 2024, 8:13 PM
F13966349: D16620.id40011.diff
Oct 16 2024, 7:40 AM
F13957340: D16620.id40017.diff
Oct 14 2024, 9:20 AM
Unknown Object (File)
Sep 15 2024, 12:41 PM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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
64

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!