From: Alexey Sandler <alexey.sandler@intl.att.com>
Date: Wed, 4 Dec 2019 14:10:34 +0000 (+0200)
Subject: Extract subscriber-name only when it is not defined
X-Git-Tag: 6.0.1~122^2
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=1df91a571e683fa069a4968954dc7dcc61467145;p=vid.git

Extract subscriber-name only when it is not defined

This solves the Recreate issue when subscriber-name is already
provided by the template, but Redux is not populated with
`service.subscribers`.

Issue-ID: VID-724

Signed-off-by: Alexey Sandler <alexey.sandler@intl.att.com>
Change-Id: Id3bfd705015270352aef674d2119fc77838a3ddb
Signed-off-by: Alexey Sandler <alexey.sandler@intl.att.com>
---

diff --git a/vid-webpack-master/cypress/support/jsonBuilders/mocks/jsons/instantiationTemplates/templates__instance_template.json b/vid-webpack-master/cypress/support/jsonBuilders/mocks/jsons/instantiationTemplates/templates__instance_template.json
index d94dbd39d..85f8582ab 100644
--- a/vid-webpack-master/cypress/support/jsonBuilders/mocks/jsons/instantiationTemplates/templates__instance_template.json
+++ b/vid-webpack-master/cypress/support/jsonBuilders/mocks/jsons/instantiationTemplates/templates__instance_template.json
@@ -1,5 +1,5 @@
 {
-  "action": "None_Create",
+  "action": "Create",
   "instanceName": "vProbe_NC_Service_DG_new_SI",
   "productFamilyId": null,
   "lcpCloudRegionId": null,
diff --git a/vid-webpack-master/src/app/drawingBoard/service-planning/drawing-board-header/drawing-board-header.component.ts b/vid-webpack-master/src/app/drawingBoard/service-planning/drawing-board-header/drawing-board-header.component.ts
index 7923313bf..8228f9531 100644
--- a/vid-webpack-master/src/app/drawingBoard/service-planning/drawing-board-header/drawing-board-header.component.ts
+++ b/vid-webpack-master/src/app/drawingBoard/service-planning/drawing-board-header/drawing-board-header.component.ts
@@ -180,7 +180,7 @@ export class DrawingBoardHeader {
     }
   }
 
-  extractOwningEntityNameAccordingtoId(id:String): string {
+  extractOwningEntityNameAccordingToId(id:String): string {
     let owningEntityName;
     _.forEach(this.store.getState().service.categoryParameters.owningEntityList,(owningEntity: OwningEntity) => {
       if (owningEntity.id === id) {
@@ -190,12 +190,20 @@ export class DrawingBoardHeader {
     return owningEntityName;
   }
 
+  private extractSubscriberNameByGlobalSubscriberId(globalSubscriberId: string) {
+    return this.store.getState().service.subscribers.find(sub => sub.id === globalSubscriberId).name;
+  }
+
   extractServiceFields(): any {
     let instanceFields : ServiceInstance;
     instanceFields = this.store.getState().service.serviceInstance[this.serviceModelId];
     if (instanceFields.action === ServiceInstanceActions.Create) {
-      instanceFields.subscriberName = this.store.getState().service.subscribers.find(sub => sub.id === instanceFields.globalSubscriberId).name;
-      instanceFields.owningEntityName = this.extractOwningEntityNameAccordingtoId(instanceFields.owningEntityId);
+      if(_.isNil(instanceFields.subscriberName)) {
+        instanceFields.subscriberName = this.extractSubscriberNameByGlobalSubscriberId(instanceFields.globalSubscriberId);
+      }
+      if (_.isNil(instanceFields.owningEntityName)) {
+        instanceFields.owningEntityName = this.extractOwningEntityNameAccordingToId(instanceFields.owningEntityId);
+      }
     }
     return _.omit(instanceFields,['optionalGroupMembersMap', 'upgradedVFMSonsCounter', 'isUpgraded', 'latestAvailableVersion']);
   }