From bca6a11c83157cc4f177ee63ef42ae0e81a588d1 Mon Sep 17 00:00:00 2001 From: mojahidi Date: Wed, 27 Dec 2017 18:09:24 +0530 Subject: Fixed sonar issues - NetworkManagerImpl Fixed all sonar issues Change-Id: I0baf7c56a68b60a8635dd88638a287098e685b31 Issue-ID: SDC-343 Signed-off-by: mojahidi --- .../impl/NetworkManagerImpl.java | 67 +++++++++------------- 1 file changed, 28 insertions(+), 39 deletions(-) (limited to 'openecomp-be/backend/openecomp-sdc-vendor-software-product-manager') diff --git a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/NetworkManagerImpl.java b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/NetworkManagerImpl.java index e6233bb79c..86d604da7c 100644 --- a/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/NetworkManagerImpl.java +++ b/openecomp-be/backend/openecomp-sdc-vendor-software-product-manager/src/main/java/org/openecomp/sdc/vendorsoftwareproduct/impl/NetworkManagerImpl.java @@ -1,9 +1,6 @@ -/*- - * ============LICENSE_START======================================================= - * SDC - * ================================================================================ - * Copyright (C) 2017 AT&T Intellectual Property. All rights reserved. - * ================================================================================ +/* + * Copyright © 2016-2017 European Support Limited + * * 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 @@ -15,7 +12,6 @@ * 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. - * ============LICENSE_END========================================================= */ package org.openecomp.sdc.vendorsoftwareproduct.impl; @@ -30,6 +26,8 @@ import org.openecomp.sdc.logging.types.LoggerErrorCode; import org.openecomp.sdc.logging.types.LoggerTragetServiceName; import org.openecomp.sdc.vendorsoftwareproduct.NetworkManager; import org.openecomp.sdc.vendorsoftwareproduct.dao.NetworkDao; +import org.openecomp.sdc.vendorsoftwareproduct.dao.VendorSoftwareProductInfoDao; +import org.openecomp.sdc.vendorsoftwareproduct.dao.VendorSoftwareProductInfoDaoFactory; import org.openecomp.sdc.vendorsoftwareproduct.dao.type.NetworkEntity; import org.openecomp.sdc.vendorsoftwareproduct.dao.type.VspDetails; import org.openecomp.sdc.vendorsoftwareproduct.errors.CompositionEditNotAllowedErrorBuilder; @@ -47,10 +45,14 @@ import org.openecomp.sdc.versioning.dao.types.Version; import java.util.Collection; public class NetworkManagerImpl implements NetworkManager { - private static MdcDataDebugMessage mdcDataDebugMessage = new MdcDataDebugMessage(); + private static final MdcDataDebugMessage MDC_DATA_DEBUG_MESSAGE = new MdcDataDebugMessage(); - private NetworkDao networkDao; - private CompositionEntityDataManager compositionEntityDataManager; + private final NetworkDao networkDao; + private final CompositionEntityDataManager compositionEntityDataManager; + private static final VendorSoftwareProductInfoDao VSP_INFO_DAO + = VendorSoftwareProductInfoDaoFactory.getInstance().createInterface(); + private static final String VSP_ID = "VSP id"; + private static final String VSP_ID_NETWORK_ID = "VSP id, network id"; public NetworkManagerImpl(NetworkDao networkDao, CompositionEntityDataManager compositionEntityDataManager) { @@ -60,17 +62,17 @@ public class NetworkManagerImpl implements NetworkManager { @Override public Collection listNetworks(String vspId, Version version) { - mdcDataDebugMessage.debugEntryMessage("VSP id", vspId); - mdcDataDebugMessage.debugExitMessage("VSP id", vspId); + MDC_DATA_DEBUG_MESSAGE.debugEntryMessage(VSP_ID, vspId); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage(VSP_ID, vspId); return networkDao.list(new NetworkEntity(vspId, version, null)); } @Override public NetworkEntity createNetwork(NetworkEntity network) { - mdcDataDebugMessage.debugEntryMessage("VSP id", network.getVspId()); + MDC_DATA_DEBUG_MESSAGE.debugEntryMessage(VSP_ID, network.getVspId()); - if (!isManual(network.getVspId(), network.getVersion())) { + if (!VSP_INFO_DAO.isManual(network.getVspId(), network.getVersion())) { MdcDataErrorMessage.createErrorMessageAndUpdateMdc(LoggerConstants.TARGET_ENTITY_DB, LoggerTragetServiceName.CREATE_NETWORK, ErrorLevel.ERROR.name(), LoggerErrorCode.PERMISSION_ERROR.getErrorCode(), "Can't create network"); @@ -79,23 +81,20 @@ public class NetworkManagerImpl implements NetworkManager { .build()); } -/* networkDao - .updateVspLatestModificationTime(network.getVspId(), network.getVersion());*/ - - mdcDataDebugMessage.debugExitMessage("VSP id", network.getVspId()); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage(VSP_ID, network.getVspId()); return null; } @Override public CompositionEntityValidationData updateNetwork(NetworkEntity network) { - mdcDataDebugMessage - .debugEntryMessage("VSP id, network id", network.getVspId(), network.getId()); + MDC_DATA_DEBUG_MESSAGE + .debugEntryMessage(VSP_ID_NETWORK_ID, network.getVspId(), network.getId()); NetworkEntity retrieved = getValidatedNetwork(network.getVspId(), network.getVersion(), network.getId()); NetworkCompositionSchemaInput schemaInput = new NetworkCompositionSchemaInput(); - schemaInput.setManual(isManual(network.getVspId(), network.getVersion())); + schemaInput.setManual(!VSP_INFO_DAO.isManual(network.getVspId(), network.getVersion())); schemaInput.setNetwork(retrieved.getNetworkCompositionData()); CompositionEntityValidationData validationData = compositionEntityDataManager @@ -104,11 +103,8 @@ public class NetworkManagerImpl implements NetworkManager { networkDao.update(network); } -/* networkDao - .updateVspLatestModificationTime(network.getVspId(), network.getVersion());*/ - - mdcDataDebugMessage - .debugExitMessage("VSP id, network id", network.getVspId(), network.getId()); + MDC_DATA_DEBUG_MESSAGE + .debugExitMessage(VSP_ID_NETWORK_ID, network.getVspId(), network.getId()); return validationData; } @@ -116,13 +112,13 @@ public class NetworkManagerImpl implements NetworkManager { @Override public CompositionEntityResponse getNetwork(String vspId, Version version, String networkId) { - mdcDataDebugMessage.debugEntryMessage("VSP id, network id", vspId, networkId); + MDC_DATA_DEBUG_MESSAGE.debugEntryMessage(VSP_ID_NETWORK_ID, vspId, networkId); NetworkEntity networkEntity = getValidatedNetwork(vspId, version, networkId); Network network = networkEntity.getNetworkCompositionData(); NetworkCompositionSchemaInput schemaInput = new NetworkCompositionSchemaInput(); - schemaInput.setManual(isManual(vspId, version)); + schemaInput.setManual(!VSP_INFO_DAO.isManual(vspId, version)); schemaInput.setNetwork(network); CompositionEntityResponse response = new CompositionEntityResponse<>(); @@ -130,7 +126,7 @@ public class NetworkManagerImpl implements NetworkManager { response.setData(network); response.setSchema(getCompositionSchema(schemaInput)); - mdcDataDebugMessage.debugExitMessage("VSP id, network id", vspId, networkId); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage(VSP_ID_NETWORK_ID, vspId, networkId); return response; } @@ -145,9 +141,9 @@ public class NetworkManagerImpl implements NetworkManager { @Override public void deleteNetwork(String vspId, Version version, String networkId) { - mdcDataDebugMessage.debugEntryMessage("VSP id, network id", vspId, networkId); + MDC_DATA_DEBUG_MESSAGE.debugEntryMessage(VSP_ID_NETWORK_ID, vspId, networkId); - if (!isManual(vspId, version)) { + if (!VSP_INFO_DAO.isManual(vspId, version)) { MdcDataErrorMessage.createErrorMessageAndUpdateMdc(LoggerConstants.TARGET_ENTITY_DB, LoggerTragetServiceName.DELETE_NETWORK, ErrorLevel.ERROR.name(), LoggerErrorCode.PERMISSION_ERROR.getErrorCode(), "Can't delete network"); @@ -155,16 +151,9 @@ public class NetworkManagerImpl implements NetworkManager { new CompositionEditNotAllowedErrorBuilder(vspId, version).build()); } - //networkDao.updateVspLatestModificationTime(vspId, version); - - mdcDataDebugMessage.debugExitMessage("VSP id, network id", vspId, networkId); + MDC_DATA_DEBUG_MESSAGE.debugExitMessage(VSP_ID_NETWORK_ID, vspId, networkId); } - // todo *************************** move to reusable place! ************************* - - private boolean isManual(String vspId, Version version) { - return false; - } protected String getCompositionSchema(NetworkCompositionSchemaInput schemaInput) { return SchemaGenerator -- cgit 1.2.3-korg