epriestley 0a069cb55a Require a viewer to load handles ... Summary: Unmuck almost all of the we-sort-of-have-viewers-some-of-the-time mess. There are a few notable cases here: - I used Omnipotent users when indexing objects for search. I think this is correct; we do policy filtering when showing results. - I cheated in a bad way in the Remarkup object rule, but fixing this requires fixing all the PhabricatorRemarkupEngine callsites (there are 85). I'll do that in the next diff. - I cheated in a few random places, like when sending mail about package edits. These aren't a big deal. Test Plan: - Grepped for all PhabricatorObjectHandleData references. - Gave them viewers. Reviewers: vrana Reviewed By: vrana CC: aran, edward Maniphest Tasks: T603 Differential Revision: https://secure.phabricator.com/D5151		2013-02-28 17:15:09 -08:00
..
config	Search options.	2013-01-11 15:28:19 -08:00
constants	Add support for differential field specifications to be indexed in search	2012-11-07 13:31:52 -08:00
controller	Require a viewer to load handles	2013-02-28 17:15:09 -08:00
engine	Require a viewer to load handles	2013-02-28 17:15:09 -08:00
index	getConfigEnv fails fast when key is not found and no default value is given.	2013-01-19 12:11:28 -08:00
management	Apply lint rules to Phabricator	2013-02-19 13:33:10 -08:00
selector	Delete license headers from files	2012-11-05 11:16:51 -08:00
storage	Delete license headers from files	2012-11-05 11:16:51 -08:00
view	Kill phutil_render_tag()	2013-02-13 16:32:13 -08:00