From 71831035554c86ae9a60fad0b883fce7269eecfc Mon Sep 17 00:00:00 2001 From: anushadasari Date: Fri, 6 Sep 2019 15:16:27 +0530 Subject: Add Semicolon ath the end In JavaScript, the semicolon (;) is optional as a statement separator, but omitting semicolons can be confusing, and lead to unexpected results because a semicolon is implicitly inserted at the end of each line. Issue-ID: VID-607 Change-Id: I9ca9d0f3c935efe83859b4d623939adce0a38465 Signed-off-by: anushadasari --- .../change-management-manual-tasks.controller.test.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/vid-app-common/src/main/webapp/app/vid/scripts/modals/change-management-manual-tasks-controller/change-management-manual-tasks.controller.test.js b/vid-app-common/src/main/webapp/app/vid/scripts/modals/change-management-manual-tasks-controller/change-management-manual-tasks.controller.test.js index deefca7e1..339717981 100644 --- a/vid-app-common/src/main/webapp/app/vid/scripts/modals/change-management-manual-tasks-controller/change-management-manual-tasks.controller.test.js +++ b/vid-app-common/src/main/webapp/app/vid/scripts/modals/change-management-manual-tasks-controller/change-management-manual-tasks.controller.test.js @@ -102,7 +102,7 @@ describe('changeManagementManualTasksController testing', () => { // when return $controller.__test_only__.loadAvailableTasks('anything') .then(() => { - expect($controller.manualTasks).toBeUndefined() + expect($controller.manualTasks).toBeUndefined(); }); }); -- cgit 1.2.3-korg From eb3d8d9cb97f9ebde71c89c86310b78bf8fd92ba Mon Sep 17 00:00:00 2001 From: anushadasari Date: Fri, 6 Sep 2019 15:24:14 +0530 Subject: Add Semicolon at the end In JavaScript, the semicolon (;) is optional as a statement separator, but omitting semicolons can be confusing, and lead to unexpected results because a semicolon is implicitly inserted at the end of each line. Issue-ID: VID-607 Change-Id: I40df40c69bf7de18eeafbf640b4ebe1d0efe6def Signed-off-by: anushadasari --- .../new-change-management.controller.js | 40 +++++++++++----------- 1 file changed, 20 insertions(+), 20 deletions(-) diff --git a/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.js b/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.js index d9bc828fe..528d3d552 100644 --- a/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.js +++ b/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.js @@ -97,7 +97,7 @@ .catch(function (error) { $log.error(error); vm.errorMsg = err; - }) + }); }; var registerVNFNamesWatcher = function () { @@ -252,16 +252,16 @@ source: vnf.availableVersions[0].requestInfo.source, suppressRollback: vnf.availableVersions[0].requestInfo.suppressRollback, requestorId: vnf.availableVersions[0].requestInfo.requestorId - } + }; if(workflowType=='Update'){ requestParametersData = { usePreload: vnf.availableVersions[0].requestParameters.usePreload - } + }; }else if(workflowType=="Replace"){ requestParametersData = { rebuildVolumeGroups: vnf.availableVersions[0].requestParameters.usePreload - } + }; }else if(workflowType=="VNF In Place Software Update"){ var payloadObj = { 'existing_software_version':vm.getInternalWorkFlowParameter(workflowType, 'text', 'Existing software version').value, @@ -270,11 +270,11 @@ }; requestParametersData = { payload: JSON.stringify(payloadObj) - } + }; }else if(workflowType=="VNF Config Update"){ requestParametersData = { payload: vm.getInternalWorkFlowParameter("VNF Config Update", "FILE", "Attach configuration file").value - } + }; }else if(workflowType=="VNF Scale Out"){ if(!moduleToScale) return null; @@ -282,12 +282,12 @@ requestParametersData = { userParams: moduleToScale.userParams //,usePreload: true - } + }; }else{ requestParametersData = { userParams: [] //,usePreload: false - } + }; } } $log.info('SchedulerWidgetCtrl:extractChangeManagementCallbackDataStr info:: workflowType '+ workflowType); @@ -301,11 +301,11 @@ }; requestParametersData = { payload: JSON.stringify(payloadObj) - } + }; }else if(workflowType=="VNF Config Update"){ requestParametersData = { payload: vm.getInternalWorkFlowParameter("VNF Config Update", "FILE", "Attach configuration file").value - } + }; } var data; @@ -525,7 +525,7 @@ const nodeType = vnfsData[i]['node-type']; if (nodeType === "generic-vnf") { if (_.find(vnfsData[i]['related-to'], function (node) { - return node['node-type'] === 'vserver' + return node['node-type'] === 'vserver'; }) !== undefined) { vm.vnfs.push(vnfsData[i]); } @@ -551,7 +551,7 @@ }); _.forEach(filteredVnfs, function (vnf) { - vm.vnfTypes.push(vnf.properties['nf-role']) + vm.vnfTypes.push(vnf.properties['nf-role']); }); } } @@ -600,7 +600,7 @@ vm.fromVNFVersions = _.uniqBy(fromVNFVersions, 'value'); } - }) + }); } }; @@ -724,7 +724,7 @@ .then(vm.loadLocalWorkFlowsParameters) .then(function () { vm.workflows = vm.localWorkflows; - }) + }); } }; @@ -780,7 +780,7 @@ let parameters = []; workflow.workflowInputParameters .filter( function (param) { - return param.soPayloadLocation === "userParams" + return param.soPayloadLocation === "userParams"; }) .forEach(function (param) { let workflowParams = vm.repackAttributes(param); @@ -809,27 +809,27 @@ soFieldName: workflowParam.soFieldName, maxLength: '500', pattern: '.*' - } + }; }; vm.getRemoteWorkFlowParameters = function (workflow) { if (workflow && vm.remoteWorkflowsParameters.has(workflow)) { - return vm.remoteWorkflowsParameters.get(workflow) + return vm.remoteWorkflowsParameters.get(workflow); } return []; }; vm.hasPatternError = function(form, itemName){ return form[itemName].$error.pattern; - } + }; vm.hasAsyncFnError = function(form, itemName){ return form[itemName].$error.validateAsyncFn; - } + }; vm.getIdFor = function(type, id, name){ return "internal-workflow-parameter-" + type + "-" + id + "-" + (name ? name.split(' ').join('-').toLowerCase() : ""); - } + }; vm.getInternalWorkFlowParameters = function (workflow, type) { if (workflow && vm.localWorkflowsParameters.has(workflow) && vm.localWorkflowsParameters.get(workflow).filter(parameter => parameter.type==type) != []) { -- cgit 1.2.3-korg From 0d6b54547b5e28591e24bbb3bf045d8b9e46f113 Mon Sep 17 00:00:00 2001 From: anushadasari Date: Fri, 6 Sep 2019 15:28:36 +0530 Subject: Add Semicolon at the end In JavaScript, the semicolon (;) is optional as a statement separator, but omitting semicolons can be confusing, and lead to unexpected results because a semicolon is implicitly inserted at the end of each line. Issue-ID: VID-607 Change-Id: Ic75affcf10f990fa507eaf738a360df603a700dc Signed-off-by: anushadasari --- .../new-change-management/new-change-management.controller.test.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.test.js b/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.test.js index b11070a9f..675c1b7c0 100644 --- a/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.test.js +++ b/vid-app-common/src/main/webapp/app/vid/scripts/modals/new-change-management/new-change-management.controller.test.js @@ -359,7 +359,7 @@ describe('Testing workFlows from SO', () => { }); test('Verify get remote workflow should call internal service for params when workflow is native', () =>{ - let getWorkflowsStub = Promise.resolve({"data": {"workflows": ["workflow 0"]}}) + let getWorkflowsStub = Promise.resolve({"data": {"workflows": ["workflow 0"]}}); let getLocalWorkflowsParametersStub = Promise.resolve({"data":{ "parameterDefinitions": [ { -- cgit 1.2.3-korg