You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@brooklyn.apache.org by iu...@apache.org on 2022/07/18 13:25:48 UTC

[brooklyn-ui] branch master updated: fix overwrite rule when adding a config key definition

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

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


The following commit(s) were added to refs/heads/master by this push:
     new c83853c8 fix overwrite rule when adding a config key definition
     new 563a359d Merge pull request #351 from algairim/fixes/blueprint-service
c83853c8 is described below

commit c83853c81c14b61254a777275d724a93d5a0a5aa
Author: Mykola Mandra <my...@cloudsoft.io>
AuthorDate: Fri Jul 15 10:25:02 2022 +0100

    fix overwrite rule when adding a config key definition
    
    Signed-off-by: Mykola Mandra <my...@cloudsoft.io>
---
 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 5d88aaf5..7fd80c96 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 ? null : configDef);
+        allConfig[key] = Object.assign({}, configDef, paramMapped, overwrite ? configDef: null);
     }
     if (!skipUpdatesDuringBatch) {
         this.miscData.set('config', Object.values(allConfig));