Github user ahgittin commented on a diff in the pull request: https://github.com/apache/brooklyn-ui/pull/112#discussion_r235935687 --- Diff: ui-modules/blueprint-composer/app/components/providers/blueprint-service.provider.js --- @@ -512,6 +514,16 @@ function BlueprintService($log, $q, $sce, paletteApi, iconGenerator, dslService) entity.miscData.set('config', allConfig); } + function addUnlistedParameterDefinitions(entity) { + let allParams = entity.miscData.get('parameters') || []; + entity.parameters.forEach((param) => { + if (!allParams.some((e) => e.name === param.name)) { + allParams.push(param); + } + }); + entity.miscData.set('parameters', allParams); + } --- End diff -- aha TY. when we add paramters via the UI (coming soon) this will be needed. okay to add with a comment to this effect in the `addUnlilstedParameters` method?
---