From 629fdcadfa911e29856f9e14852604ea6307cca7 Mon Sep 17 00:00:00 2001 From: Taka Date: Tue, 23 Jan 2018 16:41:59 -0500 Subject: fixed critical issues from sonar report Change-Id: I443db1a8104bdfbfb20ec604978d758edfe7fb44 Issue-ID: APPC-429 Signed-off-by: Taka --- .../impl/LifecycleManagerImpl.java | 27 ++++++++++++---------- 1 file changed, 15 insertions(+), 12 deletions(-) (limited to 'appc-lifecycle-management') diff --git a/appc-lifecycle-management/appc-lifecycle-management-core/src/main/java/org/onap/appc/lifecyclemanager/impl/LifecycleManagerImpl.java b/appc-lifecycle-management/appc-lifecycle-management-core/src/main/java/org/onap/appc/lifecyclemanager/impl/LifecycleManagerImpl.java index 12c48b481..70acfb7f5 100644 --- a/appc-lifecycle-management/appc-lifecycle-management-core/src/main/java/org/onap/appc/lifecyclemanager/impl/LifecycleManagerImpl.java +++ b/appc-lifecycle-management/appc-lifecycle-management-core/src/main/java/org/onap/appc/lifecyclemanager/impl/LifecycleManagerImpl.java @@ -9,15 +9,15 @@ * 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. * See the License for the specific language governing permissions and * limitations under the License. - * + * * ECOMP is a trademark and service mark of AT&T Intellectual Property. * ============LICENSE_END========================================================= */ @@ -76,23 +76,26 @@ public class LifecycleManagerImpl implements LifecycleManager{ return nextState.getStateName(); } - private StateMachine getStateMachine(String vnfType){ + private StateMachine getStateMachine(String vnfType) { + + String vnfType1 = vnfType; + if (logger.isTraceEnabled()) { - logger.trace("Entering to getNextState with vnfType = "+ vnfType); + logger.trace("Entering to getNextState with vnfType = "+ vnfType1); } - if(vnfType == null){ - vnfType = "DEFAULT"; + if(vnfType1 == null){ + vnfType1 = "DEFAULT"; } - StateMachine machine = stateMachineMap.get(vnfType); + StateMachine machine = stateMachineMap.get(vnfType1); if(machine == null){ - metadataReader = getMetadataReader(vnfType); + metadataReader = getMetadataReader(vnfType1); StateMachineMetadata metadata = metadataReader.readMetadata(); machine = StateMachineFactory.getStateMachine(metadata); - stateMachineMap.put(vnfType,machine); + stateMachineMap.put(vnfType1,machine); } - logger.trace("Exiting getStateMachine with StateMachine = " + stateMachineMap.get(vnfType).toString()); - return stateMachineMap.get(vnfType); + logger.trace("Exiting getStateMachine with StateMachine = " + stateMachineMap.get(vnfType1).toString()); + return stateMachineMap.get(vnfType1); } private StateMetaDataReader getMetadataReader(String vnfType) { -- cgit 1.2.3-korg