Page MenuHomePhabricator

Make restricted objects in commit messages work more consistently with the web UI
ClosedPublic

Authored by epriestley on Jan 1 2017, 5:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 17, 3:15 PM
Unknown Object (File)
Fri, Jan 10, 1:16 AM
Unknown Object (File)
Tue, Dec 24, 10:56 AM
Unknown Object (File)
Sat, Dec 21, 2:28 AM
Unknown Object (File)
Fri, Dec 20, 3:15 PM
Unknown Object (File)
Dec 15 2024, 12:28 AM
Unknown Object (File)
Dec 13 2024, 1:14 PM
Unknown Object (File)
Dec 5 2024, 7:34 PM
Subscribers
None

Details

Summary

Fixes T11344. In the web UI, if a field like "Subscribers" on an object (like a task) contains values you don't have permission to see, you see tokens for them (like "Restricted Project") but not their names.

Make commit messages work the same way: you see the PHID, and can remove it or leave it there, but can't see the underlying name.

(We have to render an actual PHID rather than just "Restricted Thing" because we have to be able to figure out what edit the user is actually trying to make.)

Test Plan

Interacted with a revision via the CLI that had project reviewers I couldn't see.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Jan 1 2017, 5:41 PM
This revision was automatically updated to reflect the committed changes.