Page MenuHomePhabricator

Respect repository identities when selecting author vs auditor actions
ClosedPublic

Authored by epriestley on Jun 17 2019, 5:11 PM.
Tags
None
Referenced Files
F15428822: D20581.diff
Sun, Mar 23, 10:35 PM
F15419221: D20581.diff
Fri, Mar 21, 4:33 AM
F15411080: D20581.diff
Wed, Mar 19, 8:49 AM
F15380952: D20581.id49086.diff
Fri, Mar 14, 5:07 AM
Unknown Object (File)
Mon, Mar 3, 11:47 PM
Unknown Object (File)
Feb 21 2025, 9:00 PM
Unknown Object (File)
Feb 9 2025, 7:52 AM
Unknown Object (File)
Feb 9 2025, 7:52 AM
Subscribers
None

Details

Summary

Depends on D20580. Fixes T13311. When we choose which actions to show a user, we can either show them "auditor" actions (like "raise concern") or "author" actions (like "request verification").

Currently, we don't show "author" actions if you're the author of the commit via an identity mapping, but we should. Use identity mappings where they exist.

(Because I've implemented getEffectiveAuthorPHID() in a way that requires $data be attached, it's possible this will make something throw a "DataNotAttached" exception, but: probably it won't?; and that's easy to fix if it happens.)

Test Plan

See D20580. As @alice, viewed the commit in the UI.

  • Before: got auditor actions presented to me.
  • After: got author actions.

Diff Detail

Repository
rP Phabricator
Branch
audit2
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22961
Build 31512: Run Core Tests
Build 31511: arc lint + arc unit