Page MenuHomePhabricator
Differential D17883

Restrict watching and member project display better
ClosedPublic
Actions

Authored by chad on May 15 2017, 3:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Nov 25, 6:16 AM
Unknown Object (File)
Thu, Nov 21, 12:19 PM
Unknown Object (File)
Sat, Nov 16, 5:02 PM
Unknown Object (File)
Thu, Nov 14, 7:11 AM
Unknown Object (File)
Wed, Nov 13, 7:32 AM
Unknown Object (File)
Sun, Nov 10, 8:21 AM
Unknown Object (File)
Sat, Nov 9, 1:39 PM
Unknown Object (File)
Thu, Nov 7, 11:09 AM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T12713: Can't view user profiles
Commits
rPfcebaa5097f3: (stable) Restrict watching and member project display better
rPea874a28251b: Restrict watching and member project display better
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad created this revision.May 15 2017, 3:59 PM
Herald added a subscriber: Korvin. · View Herald TranscriptMay 15 2017, 3:59 PM
Harbormaster completed remote builds in B16939: Diff 43013.May 15 2017, 4:00 PM
epriestley accepted this revision.May 15 2017, 4:00 PM
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
chad updated this revision to Diff 43014.May 15 2017, 4:02 PM
  • derp
Harbormaster completed remote builds in B16940: Diff 43014.May 15 2017, 4:03 PM
chad updated this revision to Diff 43015.May 15 2017, 4:03 PM
  • revert subproject too
Harbormaster completed remote builds in B16941: Diff 43015.May 15 2017, 4:04 PM
Closed by commit rPea874a28251b: Restrict watching and member project display better (authored by chad, committed by chad). · Explain WhyMay 15 2017, 4:04 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 43016

View Options

src/applications/project/controller/PhabricatorProjectProfileController.php

Loading...
View Options

src/applications/project/controller/PhabricatorProjectSubprojectsController.php

Loading...
View Options

src/applications/project/query/PhabricatorProjectSearchEngine.php

Loading...
View Options

src/applications/project/view/PhabricatorProjectListView.php

Loading...
Phabricator · Built by Phacility