X-Git-Url: https://osm.etsi.org/gitweb/?a=blobdiff_plain;f=src%2Fapp%2Futilities%2Fvm-migration%2FVmMigrationComponent.ts;h=b629497187c772c4a8dd0fbf50827e747c301d7c;hb=c7e646225214c583463c859f267d314fb04e829e;hp=7ac8af0cd39bff7b4946a891e3e238628514cb6c;hpb=bc5d33e8d5eba70e09b7c7d40089cd3d350e2741;p=osm%2FNG-UI.git diff --git a/src/app/utilities/vm-migration/VmMigrationComponent.ts b/src/app/utilities/vm-migration/VmMigrationComponent.ts index 7ac8af0..b629497 100644 --- a/src/app/utilities/vm-migration/VmMigrationComponent.ts +++ b/src/app/utilities/vm-migration/VmMigrationComponent.ts @@ -18,6 +18,7 @@ /** * @file Vm Migration Component */ +import { isNullOrUndefined } from 'util'; import { HttpHeaders } from '@angular/common/http'; import { Component, Injector, Input, OnInit } from '@angular/core'; import { AbstractControl, FormBuilder, FormGroup, Validators } from '@angular/forms'; @@ -28,8 +29,7 @@ import { environment } from 'environment'; import { VMMIGRATION } from 'NSInstanceModel'; import { RestService } from 'RestService'; import { SharedService } from 'SharedService'; -import { isNullOrUndefined } from 'util'; -import { VDUR, VNFInstanceDetails } from 'VNFInstanceModel'; +import { InstanceData, VDUR, VNFInstanceDetails } from 'VNFInstanceModel'; /** * Creating component @@ -65,7 +65,7 @@ export class VmMigrationComponent implements OnInit { public instanceId: string; /** Form valid on submit trigger @public */ public submitted: boolean = false; - /** Contains vduId @public */ + /** Contains vduId @public */ public vduId: {}; /** Items for countIndex @public */ public countIndex: {}[]; @@ -115,27 +115,32 @@ export class VmMigrationComponent implements OnInit { /** Getting MemberVnfIndex using NSDescriptor API @public */ public getMemberVnfIndex(): void { - const vnfInstanceData: {}[] = []; this.isLoadingResults = true; + const vnfInstanceData: {}[] = []; this.restService.getResource(environment.VNFINSTANCES_URL).subscribe((vnfInstancesData: VNFInstanceDetails[]): void => { vnfInstancesData.forEach((vnfData: VNFInstanceDetails): void => { + const vnfdRef: string = 'vnfd-ref'; + const memberIndex: string = 'member-vnf-index-ref'; + const nsrId: string = 'nsr-id-ref'; const vnfDataObj: {} = { - VNFD: vnfData['vnfd-ref'], + // eslint-disable-next-line security/detect-object-injection + VNFD: vnfData[vnfdRef], VNFInstanceId: vnfData._id, - MemberIndex: vnfData['member-vnf-index-ref'], - NS: vnfData['nsr-id-ref'] + // eslint-disable-next-line security/detect-object-injection + MemberIndex: vnfData[memberIndex], + // eslint-disable-next-line security/detect-object-injection + NS: vnfData[nsrId] }; vnfInstanceData.push(vnfDataObj); }); const nsId: string = 'NS'; + // eslint-disable-next-line security/detect-object-injection this.nsIdFilteredData = vnfInstanceData.filter((vnfdData: {}[]): boolean => vnfdData[nsId] === this.params.id); - this.nsIdFilteredData.forEach((resVNF: {}[]): void => { - const memberIndex: string = 'MemberIndex'; - const vnfinstanceID: string = 'VNFInstanceId'; + this.nsIdFilteredData.forEach((resVNF: InstanceData): void => { const assignMemberIndex: {} = { - id: resVNF[memberIndex], - vnfinstanceId: resVNF[vnfinstanceID] + id: resVNF.MemberIndex, + vnfinstanceId: resVNF.VNFInstanceId }; this.memberVnfIndex.push(assignMemberIndex); }); @@ -159,13 +164,14 @@ export class VmMigrationComponent implements OnInit { subscribe((vnfInstanceDetail: VNFInstanceDetails[]): void => { this.selectedvnfId = vnfInstanceDetail['vnfd-ref']; const VDU: string = 'vdur'; + // eslint-disable-next-line security/detect-object-injection if (vnfInstanceDetail[VDU] !== undefined) { + // eslint-disable-next-line security/detect-object-injection vnfInstanceDetail[VDU].forEach((vdu: VDUR): void => { const vnfInstanceDataObj: {} = { 'count-index': vdu['count-index'], VDU: vdu['vdu-id-ref'] - }; vnfInstanceData.push(vnfInstanceDataObj); }); @@ -173,6 +179,7 @@ export class VmMigrationComponent implements OnInit { const vduName: string = 'VDU'; this.vduId = this.vdu.filter((vdu: {}, index: number, self: {}[]): {} => index === self.findIndex((t: {}): {} => ( + // eslint-disable-next-line security/detect-object-injection t[vduName] === vdu[vduName] )) ); @@ -187,6 +194,7 @@ export class VmMigrationComponent implements OnInit { /** Getting count-index by filtering id */ public getCountIndex(id: string): void { const VDU: string = 'VDU'; + // eslint-disable-next-line security/detect-object-injection this.countIndex = this.vdu.filter((vnfdData: {}[]): boolean => vnfdData[VDU] === id); } @@ -217,7 +225,6 @@ export class VmMigrationComponent implements OnInit { }; } this.migrationInitialization(migrationPayload); - } /** Initialize the Vm Migration operation @public */ @@ -232,7 +239,9 @@ export class VmMigrationComponent implements OnInit { }; this.restService.postResource(apiURLHeader, migrationPayload).subscribe((result: {}): void => { this.activeModal.close(modalData); - this.router.navigate(['/instances/ns/history-operations/' + this.params.id]).catch(); + this.router.navigate(['/instances/ns/history-operations/' + this.params.id]).catch((): void => { + // Catch Navigation Error + }); }, (error: ERRORDATA): void => { this.restService.handleError(error, 'post'); this.isLoadingResults = false; @@ -241,6 +250,7 @@ export class VmMigrationComponent implements OnInit { /** Used to get the AbstractControl of controlName passed @private */ private getFormControl(controlName: string): AbstractControl { + // eslint-disable-next-line security/detect-object-injection return this.migrationForm.controls[controlName]; } }