From 59e446fb77ccf05aeae9b7046bd80e10124707c0 Mon Sep 17 00:00:00 2001 From: eschcam Date: Wed, 1 Feb 2023 12:17:18 +0000 Subject: Disable editing the name of existing attributes Also added a patch to fix an exception been thrown when editing existing attributes Issue-ID: SDC-3384 Signed-off-by: eschcam Change-Id: I524bad3046699baf66f407d1cf0c36394a66b623 --- .../attributes-outputs/attribute-be-model.ts | 6 +++- catalog-ui/src/app/models/attributes.ts | 34 +++++++++++----------- .../attributes/attribute-modal.component.html | 5 +++- .../attributes/attribute-modal.component.ts | 6 ++++ 4 files changed, 32 insertions(+), 19 deletions(-) (limited to 'catalog-ui/src/app') diff --git a/catalog-ui/src/app/models/attributes-outputs/attribute-be-model.ts b/catalog-ui/src/app/models/attributes-outputs/attribute-be-model.ts index 6af23a55ae..a6966cd799 100644 --- a/catalog-ui/src/app/models/attributes-outputs/attribute-be-model.ts +++ b/catalog-ui/src/app/models/attributes-outputs/attribute-be-model.ts @@ -64,7 +64,11 @@ export class AttributeBEModel { this.password = attribute.password; this.required = attribute.required; this.schema = attribute.schema; - this.schemaType = attribute.schemaType; + + if (attribute.schemaType) { + this.schemaType = attribute.schemaType; + } + this.type = attribute.type; this.uniqueId = attribute.uniqueId; this.value = attribute.value; diff --git a/catalog-ui/src/app/models/attributes.ts b/catalog-ui/src/app/models/attributes.ts index ae5eac0281..711e5b3c49 100644 --- a/catalog-ui/src/app/models/attributes.ts +++ b/catalog-ui/src/app/models/attributes.ts @@ -7,9 +7,9 @@ * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. * You may obtain a copy of the License at - * + * * http://www.apache.org/licenses/LICENSE-2.0 - * + * * Unless required by applicable law or agreed to in writing, software * distributed under the License is distributed on an "AS IS" BASIS, * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. @@ -19,14 +19,14 @@ */ 'use strict'; -import * as _ from "lodash"; -import {SchemaAttribute, SchemaAttributeGroupModel} from "./schema-attribute"; -import {AttributeOutputDetail} from "app/models/attributes-outputs/attribute-output-detail"; -import {AttributeBEModel} from "app/models/attributes-outputs/attribute-be-model"; +import { AttributeBEModel } from 'app/models/attributes-outputs/attribute-be-model'; +import { AttributeOutputDetail } from 'app/models/attributes-outputs/attribute-output-detail'; +import * as _ from 'lodash'; +import { SchemaAttribute, SchemaAttributeGroupModel } from './schema-attribute'; export class AttributesGroup { constructor(attributesObj?: AttributesGroup) { - _.forEach(attributesObj, (attributes: Array, instance) => { + _.forEach(attributesObj, (attributes: AttributeModel[], instance) => { this[instance] = []; _.forEach(attributes, (attribute: AttributeModel): void => { attribute.resourceInstanceUniqueId = instance; @@ -39,7 +39,7 @@ export class AttributesGroup { export interface IAttributeModel { - //server data + // server data uniqueId: string; name: string; _default: string; @@ -49,7 +49,7 @@ export interface IAttributeModel { status: string; value: string; parentUniqueId: string; - //custom data + // custom data resourceInstanceUniqueId: string; readonly: boolean; valueUniqueUid: string; @@ -57,7 +57,7 @@ export interface IAttributeModel { export class AttributeModel extends AttributeBEModel implements IAttributeModel { - //server data + // server data uniqueId: string; name: string; _default: string; @@ -67,7 +67,7 @@ export class AttributeModel extends AttributeBEModel implements IAttributeModel status: string; value: string; parentUniqueId: string; - //custom data + // custom data resourceInstanceUniqueId: string; readonly: boolean; valueUniqueUid: string; @@ -102,7 +102,7 @@ export class AttributeModel extends AttributeBEModel implements IAttributeModel if (!this.schema || !this.schema.property) { this.schema = new SchemaAttributeGroupModel(new SchemaAttribute()); } else { - //forcing creating new object, so editing different one than the object in the table + // forcing creating new object, so editing different one than the object in the table this.schema = new SchemaAttributeGroupModel(new SchemaAttribute(this.schema.property)); } @@ -116,21 +116,20 @@ export class AttributeModel extends AttributeBEModel implements IAttributeModel if (this._default && this.type === 'list') { this._default = '[' + this._default + ']'; } - this._default = this._default != "" && this._default != "[]" && this._default != "{}" ? this._default : null; + this._default = this._default != '' && this._default != '[]' && this._default != '{}' ? this._default : null; return JSON.stringify(this); } - public convertValueToView() { - //unwrapping value {} or [] if type is complex + // unwrapping value {} or [] if type is complex if (this._default && (this.type === 'map' || this.type === 'list') && ['[', '{'].indexOf(this._default.charAt(0)) > -1 && [']', '}'].indexOf(this._default.slice(-1)) > -1) { this._default = this._default.slice(1, -1); } - //also for value - for the modal in canvas + // also for value - for the modal in canvas if (this.value && (this.type === 'map' || this.type === 'list') && ['[', '{'].indexOf(this.value.charAt(0)) > -1 && [']', '}'].indexOf(this.value.slice(-1)) > -1) { @@ -144,6 +143,7 @@ export class AttributeModel extends AttributeBEModel implements IAttributeModel } this.readonly = undefined; this.resourceInstanceUniqueId = undefined; + return this; - }; + } } diff --git a/catalog-ui/src/app/ng2/pages/workspace/attributes/attribute-modal.component.html b/catalog-ui/src/app/ng2/pages/workspace/attributes/attribute-modal.component.html index 590217d0a5..d94fb546dc 100644 --- a/catalog-ui/src/app/ng2/pages/workspace/attributes/attribute-modal.component.html +++ b/catalog-ui/src/app/ng2/pages/workspace/attributes/attribute-modal.component.html @@ -36,7 +36,10 @@
- diff --git a/catalog-ui/src/app/ng2/pages/workspace/attributes/attribute-modal.component.ts b/catalog-ui/src/app/ng2/pages/workspace/attributes/attribute-modal.component.ts index 426ed4063e..e851bafc1d 100644 --- a/catalog-ui/src/app/ng2/pages/workspace/attributes/attribute-modal.component.ts +++ b/catalog-ui/src/app/ng2/pages/workspace/attributes/attribute-modal.component.ts @@ -32,6 +32,7 @@ export class AttributeModalComponent implements OnInit { // The current effective default value pattern public defaultValuePattern: string; public defaultValueErrorMessage: string; + public isEdit: boolean; // Attribute being Edited public attributeToEdit: AttributeModel; @@ -41,6 +42,11 @@ export class AttributeModalComponent implements OnInit { } ngOnInit() { + // Disable editing the name of existing attributes + if (this.attributeToEdit && this.attributeToEdit.name) { + this.isEdit = true; + } + this.revalidateDefaultValue(); } -- cgit 1.2.3-korg