From a8dc9814f491649e59bcfa17039615cd71477db1 Mon Sep 17 00:00:00 2001 From: MichaelMorris Date: Mon, 7 Mar 2022 12:44:23 +0000 Subject: [PATCH] Support property of type List> Signed-off-by: MichaelMorris Issue-ID: SDC-3898 Change-Id: Ifc9ea422e0eafcf84b5b7e47a9108f62925e2ea0 --- .../sdc/be/model/tosca/validators/ListValidator.java | 5 ++++- .../models/properties-inputs/derived-fe-property.ts | 19 ++++++++++++++++++- .../app/models/properties-inputs/property-fe-model.ts | 5 ++--- .../dynamic-property/dynamic-property.component.html | 5 +++-- .../services/properties.utils.ts | 9 ++++++++- 5 files changed, 35 insertions(+), 8 deletions(-) diff --git a/catalog-model/src/main/java/org/openecomp/sdc/be/model/tosca/validators/ListValidator.java b/catalog-model/src/main/java/org/openecomp/sdc/be/model/tosca/validators/ListValidator.java index a069e25f91..20d927f483 100644 --- a/catalog-model/src/main/java/org/openecomp/sdc/be/model/tosca/validators/ListValidator.java +++ b/catalog-model/src/main/java/org/openecomp/sdc/be/model/tosca/validators/ListValidator.java @@ -69,6 +69,9 @@ public class ListValidator implements PropertyTypeValidator { case JSON: innerValidator = ToscaPropertyType.JSON.getValidator(); break; + case MAP: + innerValidator = ToscaPropertyType.MAP.getValidator(); + break; default: log.debug("inner Tosca Type is unknown. {}", innerToscaType); return false; @@ -88,7 +91,7 @@ public class ListValidator implements PropertyTypeValidator { for (int i = 0; i < size; i++) { JsonElement currentValue = jo.get(i); String element = JsonUtils.toString(currentValue); - if (!innerValidator.isValid(element, null, allDataTypes)) { + if (!innerValidator.isValid(element, innerToscaType.equals(ToscaPropertyType.MAP)? ToscaPropertyType.STRING.getType(): null, allDataTypes)) { log.debug("validation of element : {} failed", element); return false; } diff --git a/catalog-ui/src/app/models/properties-inputs/derived-fe-property.ts b/catalog-ui/src/app/models/properties-inputs/derived-fe-property.ts index ac05f19b7b..2d406e210b 100644 --- a/catalog-ui/src/app/models/properties-inputs/derived-fe-property.ts +++ b/catalog-ui/src/app/models/properties-inputs/derived-fe-property.ts @@ -41,6 +41,7 @@ export class DerivedFEProperty extends PropertyBEModel { canBeDeclared: boolean; mapKey: string; mapKeyError: string; + mapInlist: boolean constructor(property: PropertyBEModel, parentName?: string, createChildOfListOrMap?: boolean, key?:string, value?:any) { if (!createChildOfListOrMap) { //creating a standard derived prop @@ -60,6 +61,11 @@ export class DerivedFEProperty extends PropertyBEModel { this.mapKey = property.schema.property.type.split('.').pop(); this.mapKeyError = null; this.type = property.schema.property.type; + if (this.type == PROPERTY_TYPES.MAP){ + this.mapInlist = true; + } + + this.schema = new SchemaPropertyGroupModel(new SchemaProperty(property.schema.property)); } else { //map if (key) { this.mapKey = key; @@ -69,9 +75,19 @@ export class DerivedFEProperty extends PropertyBEModel { this.mapKeyError = 'Key cannot be empty.'; } this.type = property.type; + if (property.schema.property.type == PROPERTY_TYPES.MAP){ + const schProp = new SchemaProperty(); + schProp.isSimpleType = true; + schProp.isDataType = false; + schProp.simpleType = PROPERTY_TYPES.STRING; + this.schema = new SchemaPropertyGroupModel(schProp); + this.schemaType = PROPERTY_TYPES.STRING; + } else { + this.schema = new SchemaPropertyGroupModel(new SchemaProperty(property.schema.property)); + } + } this.valueObj = (this.type == PROPERTY_TYPES.JSON && typeof value == 'object') ? JSON.stringify(value) : value; - this.schema = new SchemaPropertyGroupModel(new SchemaProperty(property.schema.property)); this.updateValueObjOrig(); } // this.constraints = property ? property.constraints : null; @@ -97,6 +113,7 @@ export class DerivedFEProperty extends PropertyBEModel { public hasValueObjChanged() { return !_.isEqual(this.valueObj, this.valueObjOrig); } + } export class DerivedFEPropertyMap { [parentPath: string]: Array; diff --git a/catalog-ui/src/app/models/properties-inputs/property-fe-model.ts b/catalog-ui/src/app/models/properties-inputs/property-fe-model.ts index 3b8c2d1351..d4b45408ca 100644 --- a/catalog-ui/src/app/models/properties-inputs/property-fe-model.ts +++ b/catalog-ui/src/app/models/properties-inputs/property-fe-model.ts @@ -157,7 +157,7 @@ export class PropertyFEModel extends PropertyBEModel { _.set(this.valueObj, childPropName, null); } } else { - _.set(this.valueObj, childPropName, childProp.valueObj); + _.set(this.valueObj, childPropName, childProp.valueObj); } if (childProp.valueObjIsChanged) { _.set(this.valueObjValidation, childPropName, childProp.valueObjIsValid); @@ -175,7 +175,6 @@ export class PropertyFEModel extends PropertyBEModel { if (!childProp.isChildOfListOrMap || childProp.derivedDataType !== DerivedPropertyType.MAP) { return; } - const childParentNames = this.getParentNamesArray(childProp.parentName); const oldActualMapKey = childProp.getActualMapKey(); @@ -240,7 +239,7 @@ export class PropertyFEModel extends PropertyBEModel { let nameToInsert: string = parentProp.name; if (parentProp.isChildOfListOrMap) { - if (!noHashKeys && parentProp.derivedDataType == DerivedPropertyType.MAP) { + if (!noHashKeys && parentProp.derivedDataType == DerivedPropertyType.MAP && !parentProp.mapInlist) { nameToInsert = parentProp.getActualMapKey(); } else { //LIST let siblingProps = this.flattenedChildren.filter(prop => prop.parentName == parentProp.parentName).map(prop => prop.propertiesName); diff --git a/catalog-ui/src/app/ng2/components/logic/properties-table/dynamic-property/dynamic-property.component.html b/catalog-ui/src/app/ng2/components/logic/properties-table/dynamic-property/dynamic-property.component.html index 78c3030842..81f0d533b4 100644 --- a/catalog-ui/src/app/ng2/components/logic/properties-table/dynamic-property/dynamic-property.component.html +++ b/catalog-ui/src/app/ng2/components/logic/properties-table/dynamic-property/dynamic-property.component.html @@ -27,7 +27,7 @@
{{property.name}}
-
+
- Add value to list + Add value to list + Add value to map diff --git a/catalog-ui/src/app/ng2/pages/properties-assignment/services/properties.utils.ts b/catalog-ui/src/app/ng2/pages/properties-assignment/services/properties.utils.ts index 34d2a53afd..02dbf91868 100644 --- a/catalog-ui/src/app/ng2/pages/properties-assignment/services/properties.utils.ts +++ b/catalog-ui/src/app/ng2/pages/properties-assignment/services/properties.utils.ts @@ -129,7 +129,14 @@ export class PropertiesUtils { if (property.derivedDataType == DerivedPropertyType.LIST || property.derivedDataType == DerivedPropertyType.MAP) { property.flattenedChildren = []; Object.keys(property.valueObj).forEach((key) => { - property.flattenedChildren.push(...this.createListOrMapChildren(property, key, property.valueObj[key])) + property.flattenedChildren.push(...this.createListOrMapChildren(property, key, property.valueObj[key])); + const lastCreatedChild = property.flattenedChildren.slice(-1)[0]; + if (property.schemaType == PROPERTY_TYPES.MAP && property.valueObj[key]){ + const nestedValue:object = property.valueObj[key]; + Object.keys(nestedValue).forEach((keyNested) => { + property.flattenedChildren.push(...this.createListOrMapChildren(lastCreatedChild, keyNested, nestedValue[keyNested])); + }); + }; }); } else if (property.derivedDataType === DerivedPropertyType.COMPLEX) { property.flattenedChildren = this.createFlattenedChildren(property.type, property.name); -- 2.16.6