Paths

Table of Contentst

Differential D12343

Document just-in-time methods for handles
ClosedPublic
Actions

Authored by epriestley on Apr 9 2015, 11:10 PM.

Tags

None

Referenced Files

	F19554070: D12343.diff
	Thu, Jan 29, 5:09 PM

	F19554061: D12343.diff
	Thu, Jan 29, 5:06 PM

	F19544830: D12343.diff
	Fri, Jan 23, 5:26 PM

	F19534910: D12343.id29652.diff
	Wed, Jan 21, 3:19 AM

	F19502706: D12343.diff
	Thu, Jan 8, 6:34 PM

	F19502698: D12343.diff
	Thu, Jan 8, 6:28 PM

	F19502363: D12343.diff
	Thu, Jan 8, 4:14 PM

	F19498861: D12343.diff
	Thu, Jan 8, 1:19 PM

Subscribers

Details

Reviewers

Maniphest Tasks

T7689: Load handles just-in-time

Commits

Restricted Diffusion Commit
rP3c254469c95c: Document just-in-time methods for handles

Summary

Fixes T7689. I'm not going to go clean up all the rest of the loadViewerHandles() calls right now since a lot of them are kind of a pain and they aren't really hurting anything so it doesn't feel very leveraged, but at least deprecate it and document the new hotness.

Test Plan

have a look
in a book
reading rainbow

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 29652.Apr 9 2015, 11:10 PM

epriestley retitled this revision from to Document just-in-time methods for handles.

epriestley updated this object.

epriestley edited the test plan for this revision. (Show Details)

epriestley added a reviewer: btrahan.

epriestley added a task: T7689: Load handles just-in-time.

Herald added a subscriber: epriestley. · View Herald TranscriptApr 9 2015, 11:10 PM

btrahan accepted this revision.Apr 10 2015, 5:13 PM

btrahan edited edge metadata.

This revision is now accepted and ready to land.Apr 10 2015, 5:13 PM

Closed by commit rP3c254469c95c: Document just-in-time methods for handles (authored by epriestley, committed by epriestley). · Explain WhyApr 10 2015, 5:17 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

base/

controller/

PhabricatorController.php

5 lines

docs/

tech/

handles.diviner

101 lines

Diff 29659

src/applications/base/controller/PhabricatorController.php

Loading...

src/docs/tech/handles.diviner

Loading...