Purpose

The following node preview for a Dictionary in a CBN containing multple output ports does not expand currently. This is the screenshot after the fix:

This is because preferredDictionaryOrdering was a list of empty strings (3 in this case, one for each output port) and even though it should have been ignored it passed the check in DefaultWatchHandler::ProcessThing. This fix it to prevent the list from containing empty strings. Note that preferredDictionaryOrdering must only be populated for multi output nodes that have a MultiReturnAttribute.

Declarations

Check these if you believe they are true

• The code base is in a better state after this PR
• Is documented according to the standards
• The level of testing this PR includes is appropriate
• User facing strings, if any, are extracted into *.resx files
• All tests pass using the self-service CI.
• Snapshot of UI changes, if any.
• Changes to the API follow Semantic Versioning, and are documented in the API Changes document.

Reviewers

@mjkkirschner

FYIs

@jnealb