Paths

Table of Contentst

Differential D19876

Remove application callsites to "LiskDAO->loadOneRelative()"
ClosedPublic
Actions

Authored by epriestley on Dec 12 2018, 11:49 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sat, Dec 21, 1:51 PM

	Unknown Object (File)
	Mon, Dec 16, 4:49 AM

	Unknown Object (File)
	Mon, Dec 16, 4:49 AM

	Unknown Object (File)
	Mon, Dec 16, 4:49 AM

	Unknown Object (File)
	Mon, Dec 16, 4:49 AM

	Unknown Object (File)
	Sun, Dec 8, 11:04 PM

	Unknown Object (File)
	Wed, Dec 4, 11:45 PM

	Unknown Object (File)
	Sun, Dec 1, 1:12 AM

Subscribers

Restricted Owners Package

Details

Reviewers

Maniphest Tasks

T13218: Remove LiskDAO->loadOneRelative()

Commits

rP793f185d2924: Remove application callsites to "LiskDAO->loadOneRelative()"

Summary

Ref T13218. This is like loadOneWhere(...) but with more dark magic. Get rid of it.

Test Plan

Forced 20130219.commitsummarymig.php to hit this code and ran it with bin/storage upgrade --force --apply ....
Ran 20130409.commitdrev.php with bin/storage upgrade --force --apply ....
Called user.search to indirectly get primary email information.
Did not test Releeph at all.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Dec 12 2018, 11:49 PM

Owners added a subscriber: Restricted Owners Package.Dec 12 2018, 11:49 PM

Harbormaster completed remote builds in B21315: Diff 47464.Dec 12 2018, 11:51 PM

epriestley requested review of this revision.Dec 12 2018, 11:51 PM

amckinley accepted this revision.Dec 12 2018, 11:57 PM

This revision is now accepted and ready to land.Dec 12 2018, 11:57 PM

Closed by commit rP793f185d2924: Remove application callsites to "LiskDAO->loadOneRelative()" (authored by epriestley). · Explain WhyDec 13 2018, 12:39 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

sql/

patches/

20130219.commitsummarymig.php

6 lines

20130409.commitdrev.php

4 lines

src/

applications/

people/

storage/

PhabricatorUser.php

11 lines

releeph/

conduit/

ReleephGetBranchesConduitAPIMethod.php

7 lines

storage/

ReleephRequest.php

13 lines

Diff 47472

resources/sql/patches/20130219.commitsummarymig.php

Loading...

resources/sql/patches/20130409.commitdrev.php

Loading...

src/applications/people/storage/PhabricatorUser.php

Loading...

src/applications/releeph/conduit/ReleephGetBranchesConduitAPIMethod.php

Loading...

src/applications/releeph/storage/ReleephRequest.php

Loading...