Github user tbouron commented on a diff in the pull request:

    https://github.com/apache/brooklyn-ui/pull/112#discussion_r235510831
  
    --- 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 --
    
    I don't understand what this method is doing. It seems that you compare 
`entity.miscData.get('parameters')` and `entity.parameters` but these are the 
same.


---

Reply via email to