Page MenuHomePhabricator

Policy - clean up access to user profile image uri
AbandonedPublic

Authored by btrahan on Feb 2 2015, 9:58 PM.
Tags
None
Referenced Files
F17865509: D11631.id27988.diff
Mon, Jul 28, 3:25 AM
F17863894: D11631.id27988.diff
Mon, Jul 28, 1:34 AM
F17862831: D11631.id27988.diff
Mon, Jul 28, 12:21 AM
F17855827: D11631.id.diff
Sun, Jul 27, 4:36 PM
F17845604: D11631.diff
Sun, Jul 27, 3:45 AM
F17629424: D11631.id27988.diff
Jul 10 2025, 8:51 AM
Unknown Object (File)
Jun 4 2025, 6:32 AM
Unknown Object (File)
Apr 9 2025, 1:33 AM
Subscribers

Details

Reviewers
epriestley
Maniphest Tasks
T7094: Clean up T603
Summary

Ref T7094. We already had and were mostly using "needProfileImage" on the people query class. Only real trick in this diff is deleting a conduit end point that has been marked deprecated for the better part of 3 years.

Also includes a small bonus lint fix by adding the $for_app parameter to a child class so as to be consistent with the parent.

Test Plan

clicked around the people action and profiles and calendars loaded nicely.

Diff Detail

Repository
rP Phabricator
Branch
T7094_8
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 4256
Build 4269: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan retitled this revision from to Policy - clean up access to user profile image uri.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.