diff options
author | Liang Ding <liang.ding@intel.com> | 2019-08-21 21:58:54 -0700 |
---|---|---|
committer | Liang Ding <liang.ding@intel.com> | 2019-09-06 11:55:52 +0000 |
commit | a30778c360c0bcb8051e395e0ac4e9a3225c903a (patch) | |
tree | d3aad47794569838d9a949af0b647d787dfb2238 | |
parent | 06b4321bfa2a0aa5dd7529205b557ef587647c94 (diff) |
fix a nested node type issue
the newly added node type is considered as nested
when substitution_mapping is enabled in TOSCA yaml file
Change-Id: Ibdee29d7a553c7b1ce085cbe971ceace6c4488f5
Issue-ID: SDC-2282
Signed-off-by: Liang Ding <liang.ding@intel.com>
-rw-r--r-- | catalog-be/src/main/java/org/openecomp/sdc/be/components/csar/CsarInfo.java | 5 | ||||
-rw-r--r-- | catalog-model/src/main/java/org/openecomp/sdc/be/model/NodeTypeInfo.java | 7 |
2 files changed, 11 insertions, 1 deletions
diff --git a/catalog-be/src/main/java/org/openecomp/sdc/be/components/csar/CsarInfo.java b/catalog-be/src/main/java/org/openecomp/sdc/be/components/csar/CsarInfo.java index 72efb1176f..1d49394b17 100644 --- a/catalog-be/src/main/java/org/openecomp/sdc/be/components/csar/CsarInfo.java +++ b/catalog-be/src/main/java/org/openecomp/sdc/be/components/csar/CsarInfo.java @@ -167,6 +167,7 @@ public class CsarInfo { private ResultStatusEnum handleSubstitutionMappings(Map<String, NodeTypeInfo> nodeTypesInfo, Map.Entry<String, byte[]> entry, Map<String, Object> mappedToscaTemplate, Map<String, Object> substitutionMappings) { if (substitutionMappings.containsKey(TypeUtils.ToscaTagNamesEnum.NODE_TYPE.getElementName())) { NodeTypeInfo nodeTypeInfo = new NodeTypeInfo(); + nodeTypeInfo.setSubstitutionMapping(true); nodeTypeInfo.setType( (String) substitutionMappings.get(TypeUtils.ToscaTagNamesEnum.NODE_TYPE.getElementName())); nodeTypeInfo.setTemplateFileName(entry.getKey()); @@ -228,7 +229,9 @@ public class CsarInfo { String type = (String) nodeTemplate.get(TypeUtils.ToscaTagNamesEnum.TYPE.getElementName()); if (nodeTypesInfo.containsKey(type)) { NodeTypeInfo nodeTypeInfo = nodeTypesInfo.get(type); - nodeTypeInfo.setNested(true); + if (nodeTypeInfo.isSubstitutionMapping() && type.contains(Constants.USER_DEFINED_RESOURCE_NAMESPACE_PREFIX)) { + nodeTypeInfo.setNested(true); + } } } } diff --git a/catalog-model/src/main/java/org/openecomp/sdc/be/model/NodeTypeInfo.java b/catalog-model/src/main/java/org/openecomp/sdc/be/model/NodeTypeInfo.java index 5026fff129..817060e309 100644 --- a/catalog-model/src/main/java/org/openecomp/sdc/be/model/NodeTypeInfo.java +++ b/catalog-model/src/main/java/org/openecomp/sdc/be/model/NodeTypeInfo.java @@ -28,6 +28,7 @@ public class NodeTypeInfo { String templateFileName; List<String> derivedFrom; boolean isNested; + boolean isSubstitutionMapping; Map<String, Object> mappedToscaTemplate; public NodeTypeInfo getUnmarkedCopy(){ @@ -64,6 +65,12 @@ public class NodeTypeInfo { public void setNested(boolean isNested) { this.isNested = isNested; } + public boolean isSubstitutionMapping() { + return isSubstitutionMapping; + } + public void setSubstitutionMapping(boolean isSubstitutionMapping) { + this.isSubstitutionMapping = isSubstitutionMapping; + } public Map<String, Object> getMappedToscaTemplate() { return mappedToscaTemplate; |