Fix static list value issue

Change-Id: I976404e172aae756248cb4e3fe367e1c00c72168
Issue-ID: SDC-2355
Signed-off-by: talio <tali.orenbach@amdocs.com>

diff --git a/catalog-be/pom.xml b/catalog-be/pom.xml
index e32d514..dcd4c58 100644 (file)
--- a/catalog-be/pom.xml
+++ b/catalog-be/pom.xml
@@ -624,6 +624,11 @@
             <artifactId>jackson-mapper-asl</artifactId>
             <version>${jackson.mapper.version}</version>
         </dependency>
+        <dependency>
+            <groupId>org.apache.commons</groupId>
+            <artifactId>commons-collections4</artifactId>
+            <version>${commons.collections.version}</version>
+        </dependency>
     </dependencies>
 
     <build>

diff --git a/catalog-be/src/main/java/org/openecomp/sdc/be/components/impl/ServiceBusinessLogic.java b/catalog-be/src/main/java/org/openecomp/sdc/be/components/impl/ServiceBusinessLogic.java
index a586e23..cad9a92 100644 (file)
--- a/catalog-be/src/main/java/org/openecomp/sdc/be/components/impl/ServiceBusinessLogic.java
+++ b/catalog-be/src/main/java/org/openecomp/sdc/be/components/impl/ServiceBusinessLogic.java
@@ -700,6 +700,7 @@ public class ServiceBusinessLogic extends ComponentBusinessLogic {
                } else {
                        PropertyDefinition propertyDefinition = new PropertyDefinition();
                        propertyDefinition.setType(operationInputDefinition.getType());
+                       propertyDefinition.setSchema(operationInputDefinition.getSchema());
                        propertyDefinition.setValue(value);
                        propertyDefinition.setName(operationInputDefinition.getName());
 

diff --git a/catalog-be/src/main/java/org/openecomp/sdc/be/datamodel/utils/PropertyValueConstraintValidationUtil.java b/catalog-be/src/main/java/org/openecomp/sdc/be/datamodel/utils/PropertyValueConstraintValidationUtil.java
index 073e469..72342e6 100644 (file)
--- a/catalog-be/src/main/java/org/openecomp/sdc/be/datamodel/utils/PropertyValueConstraintValidationUtil.java
+++ b/catalog-be/src/main/java/org/openecomp/sdc/be/datamodel/utils/PropertyValueConstraintValidationUtil.java
@@ -106,13 +106,13 @@ public class PropertyValueConstraintValidationUtil {
             if (Objects.nonNull(propertyDefinition)) {
                                if (ToscaType.isPrimitiveType(propertyDefinition.getType())) {
                                        propertyDefinition.setConstraints(
-                                                       org.openecomp.sdc.be.dao.utils.CollectionUtils.merge(propertyDefinition.getConstraints(),
-                                                                       dataTypeDefinitionCache.get(propertyDefinition.getType()).getConstraints()));
+                                                       org.openecomp.sdc.be.dao.utils.CollectionUtils.merge(propertyDefinition.safeGetConstraints(),
+                                                                       dataTypeDefinitionCache.get(propertyDefinition.getType()).safeGetConstraints()));
                                        evaluateConstraintsOnProperty(propertyDefinition);
                                } else if (ToscaType.isCollectionType(propertyDefinition.getType())) {
                                        propertyDefinition.setConstraints(
-                                                       org.openecomp.sdc.be.dao.utils.CollectionUtils.merge(propertyDefinition.getConstraints(),
-                                                                       dataTypeDefinitionCache.get(propertyDefinition.getType()).getConstraints()));
+                                                       org.openecomp.sdc.be.dao.utils.CollectionUtils.merge(propertyDefinition.safeGetConstraints(),
+                                                                       dataTypeDefinitionCache.get(propertyDefinition.getType()).safeGetConstraints()));
                                        evaluateConstraintsOnProperty(propertyDefinition);
                                        evaluateCollectionTypeProperties(propertyDefinition);
                                } else {

diff --git a/catalog-model/src/main/java/org/openecomp/sdc/be/model/DataTypeDefinition.java b/catalog-model/src/main/java/org/openecomp/sdc/be/model/DataTypeDefinition.java
index 4dc952d..a54e326 100644 (file)
--- a/catalog-model/src/main/java/org/openecomp/sdc/be/model/DataTypeDefinition.java
+++ b/catalog-model/src/main/java/org/openecomp/sdc/be/model/DataTypeDefinition.java
@@ -20,6 +20,7 @@
 
 package org.openecomp.sdc.be.model;
 
+import org.openecomp.sdc.be.dao.utils.CollectionUtils;
 import org.openecomp.sdc.be.datatypes.elements.DataTypeDataDefinition;
 import org.openecomp.sdc.be.datatypes.elements.PropertyDataDefinition;
 
@@ -55,6 +56,10 @@ public class DataTypeDefinition extends DataTypeDataDefinition {
         return constraints;
     }
 
+    public List<PropertyConstraint> safeGetConstraints() {
+        return CollectionUtils.safeGetList(constraints);
+    }
+
     public void setConstraints(List<PropertyConstraint> constraints) {
         this.constraints = constraints;
     }