This is an automated email from the ASF dual-hosted git repository.

heneveld pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/brooklyn-ui.git

commit 1b9c86b00410753ca8c022c6cba4930550098811
Author: Alex Heneveld <[email protected]>
AuthorDate: Wed Nov 30 13:26:11 2022 +0000

    fix issue where config keys can be cleared/overridden when simply setting 
them
---
 ui-modules/blueprint-composer/app/components/util/model/entity.model.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/ui-modules/blueprint-composer/app/components/util/model/entity.model.js 
b/ui-modules/blueprint-composer/app/components/util/model/entity.model.js
index faa260a0..ba2dbc86 100644
--- a/ui-modules/blueprint-composer/app/components/util/model/entity.model.js
+++ b/ui-modules/blueprint-composer/app/components/util/model/entity.model.js
@@ -754,7 +754,7 @@ function addConfigKeyDefinition(param, overwrite, 
skipUpdatesDuringBatch, value)
             delete paramMapped['default'];
         }
         // making sure we assign to empty object to avoid conflicts with 
config items' properties like `type`
-        allConfig[key] = Object.assign({}, configDef, paramMapped, overwrite ? 
configDef: null);
+        allConfig[key] = Object.assign({}, configDef, paramMapped, overwrite ? 
null: configDef);
     }
     if (!skipUpdatesDuringBatch) {
         this.miscData.set('config', Object.values(allConfig));

Reply via email to