Correct issue with "bindings" conduit attachment
ClosedPublic
Actions

Authored by epriestley on Mar 27 2021, 4:25 PM.

Tags

None

Referenced Files

	F15193477: D21657.id.diff
	Sat, Feb 22, 5:35 AM

	F15192402: D21657.id51531.diff
	Sat, Feb 22, 1:12 AM

	F15189025: D21657.id51532.diff
	Fri, Feb 21, 3:29 PM

	Unknown Object (File)
	Wed, Feb 19, 6:56 PM

	Unknown Object (File)
	Tue, Jan 28, 5:49 AM

	Unknown Object (File)
	Tue, Jan 28, 5:49 AM

	Unknown Object (File)
	Tue, Jan 28, 5:49 AM

	Unknown Object (File)
	Mon, Jan 27, 12:51 PM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13641: Support "Disabled" devices in Almanac
Commits: rP33c847e667b6: (stable) Correct issue with "bindings" conduit attachment
rP5b8b5f214148: Correct issue with "bindings" conduit attachment

Summary

Ref T13641. These conditions are swapped, and "activeBindings" loads more data than necessary while "bindings" doesn't load enough.

Test Plan

Called method with each attachment, got good results instead of an exception.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Mar 27 2021, 4:25 PM

Harbormaster completed remote builds in B25337: Diff 51531.Mar 27 2021, 4:26 PM

This revision was not accepted when it landed; it landed in state Needs Review.Mar 27 2021, 4:26 PM

epriestley requested review of this revision.

Closed by commit rP5b8b5f214148: Correct issue with "bindings" conduit attachment (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rP5b8b5f214148: Correct issue with "bindings" conduit attachment.

epriestley added a commit: rP33c847e667b6: (stable) Correct issue with "bindings" conduit attachment.

Revision Contents
Changeset List

Path

Size

src/

applications/

almanac/

engineextension/

AlmanacBindingsSearchEngineAttachment.php

4 lines

Diff 51532

View Options

src/applications/almanac/engineextension/AlmanacBindingsSearchEngineAttachment.php

Correct issue with "bindings" conduit attachmentClosedPublicActions