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
F18876052: D11631.id.diff
Nov 5 2025, 9:19 PM
F18868339: D11631.diff
Nov 4 2025, 6:01 AM
F18813936: D11631.diff
Oct 20 2025, 8:10 PM
F18733100: D11631.id.diff
Sep 30 2025, 9:25 PM
F18626813: D11631.diff
Sep 16 2025, 2:33 AM
F18427063: D11631.id.diff
Aug 30 2025, 11:39 PM
F18389770: D11631.diff
Aug 29 2025, 6:08 AM
F18381286: D11631.diff
Aug 28 2025, 7:31 PM
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.