Correct issue with "bindings" conduit attachment
ClosedPublic
Actions

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

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.

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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.

This revision was automatically updated to reflect the committed changes.

Path

Size

src/

applications/

almanac/

engineextension/

4 lines