X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=renderer%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Ftransportpce%2Frenderer%2Fopenroadminterface%2FOpenRoadmInterfaceFactory.java;h=41e49fd5078b2eef8a019e1df5ce6d91a4eb891d;hb=refs%2Fchanges%2F48%2F89448%2F5;hp=cf8380386d3d8d26cf74ac006cb6f49201cc6843;hpb=5d5da5eeecc7b6a775c8f5a0d1b5a7062a176d63;p=transportpce.git diff --git a/renderer/src/main/java/org/opendaylight/transportpce/renderer/openroadminterface/OpenRoadmInterfaceFactory.java b/renderer/src/main/java/org/opendaylight/transportpce/renderer/openroadminterface/OpenRoadmInterfaceFactory.java index cf8380386..41e49fd50 100644 --- a/renderer/src/main/java/org/opendaylight/transportpce/renderer/openroadminterface/OpenRoadmInterfaceFactory.java +++ b/renderer/src/main/java/org/opendaylight/transportpce/renderer/openroadminterface/OpenRoadmInterfaceFactory.java @@ -9,7 +9,6 @@ package org.opendaylight.transportpce.renderer.openroadminterface; import java.util.List; - import org.opendaylight.transportpce.common.StringConstants; import org.opendaylight.transportpce.common.device.DeviceTransactionManager; import org.opendaylight.transportpce.common.mapping.MappingUtils; @@ -21,6 +20,8 @@ import org.slf4j.LoggerFactory; public class OpenRoadmInterfaceFactory { + private static final String OTN_FUNTIONS_ARE_NOT_SUPPORTED_BY_OPENROADM_MODELS_1_2_1_MSG = + "OTN funtions are not supported by Openroadm models 1.2.1"; private static final Logger LOG = LoggerFactory.getLogger(OpenRoadmInterfaceFactory.class); private final MappingUtils mappingUtils; private final OpenRoadmInterface121 openRoadmInterface121; @@ -161,7 +162,7 @@ public class OpenRoadmInterfaceFactory { } } - public boolean isUsedbyXc(String nodeId, String interfaceName, String xc, + public boolean isUsedByXc(String nodeId, String interfaceName, String xc, DeviceTransactionManager deviceTransactionManager) { switch (mappingUtils.getOpenRoadmVersion(nodeId)) { case StringConstants.OPENROADM_DEVICE_VERSION_1_2_1: @@ -173,11 +174,11 @@ public class OpenRoadmInterfaceFactory { } } - public boolean isUsedbyOtnXc(String nodeId, String interfaceName, String xc, + public boolean isUsedByOtnXc(String nodeId, String interfaceName, String xc, DeviceTransactionManager deviceTransactionManager) { switch (mappingUtils.getOpenRoadmVersion(nodeId)) { case StringConstants.OPENROADM_DEVICE_VERSION_1_2_1: - LOG.error("OTN funtions are not supported by Openroadm models 1.2.1"); + LOG.error(OTN_FUNTIONS_ARE_NOT_SUPPORTED_BY_OPENROADM_MODELS_1_2_1_MSG); return false; case StringConstants.OPENROADM_DEVICE_VERSION_2_2_1: return openRoadmInterface221.isUsedByOtnXc(nodeId, interfaceName, xc, deviceTransactionManager); @@ -190,7 +191,7 @@ public class OpenRoadmInterfaceFactory { String logicalConnPoint) throws OpenRoadmInterfaceException { switch (mappingUtils.getOpenRoadmVersion(nodeId)) { case StringConstants.OPENROADM_DEVICE_VERSION_1_2_1: - LOG.error("OTN funtions are not supported by Openroadm models 1.2.1"); + LOG.error(OTN_FUNTIONS_ARE_NOT_SUPPORTED_BY_OPENROADM_MODELS_1_2_1_MSG); return null; case StringConstants.OPENROADM_DEVICE_VERSION_2_2_1: return openRoadmOtnInterface.createOpenRoadmEth1GInterface(nodeId, logicalConnPoint); @@ -203,7 +204,7 @@ public class OpenRoadmInterfaceFactory { String logicalConnPoint) throws OpenRoadmInterfaceException { switch (mappingUtils.getOpenRoadmVersion(nodeId)) { case StringConstants.OPENROADM_DEVICE_VERSION_1_2_1: - LOG.error("OTN funtions are not supported by Openroadm models 1.2.1"); + LOG.error(OTN_FUNTIONS_ARE_NOT_SUPPORTED_BY_OPENROADM_MODELS_1_2_1_MSG); return null; case StringConstants.OPENROADM_DEVICE_VERSION_2_2_1: return openRoadmOtnInterface.createOpenRoadmEth10GInterface(nodeId, logicalConnPoint); @@ -218,7 +219,7 @@ public class OpenRoadmInterfaceFactory { throws OpenRoadmInterfaceException { switch (mappingUtils.getOpenRoadmVersion(nodeId)) { case StringConstants.OPENROADM_DEVICE_VERSION_1_2_1: - LOG.error("OTN funtions are not supported by Openroadm models 1.2.1"); + LOG.error(OTN_FUNTIONS_ARE_NOT_SUPPORTED_BY_OPENROADM_MODELS_1_2_1_MSG); return null; case StringConstants.OPENROADM_DEVICE_VERSION_2_2_1: return openRoadmOtnInterface.createOpenRoadmOdu0Interface( @@ -233,7 +234,7 @@ public class OpenRoadmInterfaceFactory { throws OpenRoadmInterfaceException { switch (mappingUtils.getOpenRoadmVersion(nodeId)) { case StringConstants.OPENROADM_DEVICE_VERSION_1_2_1: - LOG.error("OTN funtions are not supported by Openroadm models 1.2.1"); + LOG.error(OTN_FUNTIONS_ARE_NOT_SUPPORTED_BY_OPENROADM_MODELS_1_2_1_MSG); return null; case StringConstants.OPENROADM_DEVICE_VERSION_2_2_1: return openRoadmOtnInterface.createOpenRoadmOdu2Interface( @@ -248,7 +249,7 @@ public class OpenRoadmInterfaceFactory { throws OpenRoadmInterfaceException { switch (mappingUtils.getOpenRoadmVersion(nodeId)) { case StringConstants.OPENROADM_DEVICE_VERSION_1_2_1: - LOG.error("OTN funtions are not supported by Openroadm models 1.2.1"); + LOG.error(OTN_FUNTIONS_ARE_NOT_SUPPORTED_BY_OPENROADM_MODELS_1_2_1_MSG); return null; case StringConstants.OPENROADM_DEVICE_VERSION_2_2_1: return openRoadmOtnInterface.createOpenRoadmOdu2eInterface(