Page MenuHomePhabricator

Restrict watching and member project display better
ClosedPublic

Authored by chad on May 15 2017, 3:59 PM.
Tags
None
Referenced Files
F15411273: D17883.diff
Wed, Mar 19, 8:57 AM
F15411209: D17883.diff
Wed, Mar 19, 8:54 AM
F15411129: D17883.diff
Wed, Mar 19, 8:51 AM
F15411006: D17883.diff
Wed, Mar 19, 8:46 AM
F15410933: D17883.diff
Wed, Mar 19, 8:43 AM
F15409856: D17883.diff
Wed, Mar 19, 5:43 AM
F15409430: D17883.diff
Wed, Mar 19, 4:30 AM
F15409107: D17883.diff
Wed, Mar 19, 3:19 AM
Subscribers

Details

Summary

Fixes T12713. We don't need to show watching and member info on other views other than ApplicationSearch (for now) so add a few methods to restrict the calls.

Test Plan

Visit project search, profile, project home, project home with subprojects

Diff Detail

Repository
rP Phabricator
Branch
profile-users (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16940
Build 22638: Run Core Tests
Build 22637: arc lint + arc unit

Event Timeline

epriestley added inline comments.
src/applications/people/controller/PhabricatorPeopleProfileViewController.php
107–108 ↗(On Diff #43013)

We could get rid of this and revert D17882 now, right?

This revision is now accepted and ready to land.May 15 2017, 4:00 PM
This revision was automatically updated to reflect the committed changes.