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

	F14060587: D12343.diff
	Mon, Nov 18, 2:16 AM

	F14050843: D12343.diff
	Thu, Nov 14, 8:57 PM

	F14038426: D12343.diff
	Sun, Nov 10, 11:35 PM

	F14037212: D12343.diff
	Sun, Nov 10, 2:23 PM

	F14019008: D12343.diff
	Tue, Nov 5, 8:55 PM

	F14006434: D12343.diff
	Mon, Oct 28, 9:36 AM

	F14003678: D12343.diff
	Sat, Oct 26, 9:14 AM

	F13985699: D12343.id.diff
	Sun, Oct 20, 10:24 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...