diff --git a/org.opentosca.bus/org.opentosca.bus.management.service/src/main/java/org/opentosca/bus/management/service/impl/util/Util.java b/org.opentosca.bus/org.opentosca.bus.management.service/src/main/java/org/opentosca/bus/management/service/impl/util/Util.java index 2493ddfc7..9400a1be6 100644 --- a/org.opentosca.bus/org.opentosca.bus.management.service/src/main/java/org/opentosca/bus/management/service/impl/util/Util.java +++ b/org.opentosca.bus/org.opentosca.bus.management.service/src/main/java/org/opentosca/bus/management/service/impl/util/Util.java @@ -17,70 +17,74 @@ public class Util { - private final static Logger LOG = LoggerFactory.getLogger(Util.class); + private final static Logger LOG = LoggerFactory.getLogger(Util.class); - /** - * Determine the ServiceTemplateInstanceId long from the ServiceInstanceId QName. - * - * @return the ServiceTemplateInstanceId if the retrieval is successful, Long.MIN_VALUE otherwise - */ - public static long determineServiceTemplateInstanceId(final URI serviceInstanceID) { - if (Objects.nonNull(serviceInstanceID)) { - try { - if (serviceInstanceID.toString().contains("/")) { - return Long.parseLong(StringUtils.substringAfterLast(serviceInstanceID.toString(), "/")); - } else { - return Long.parseLong(serviceInstanceID.toString()); - } - } catch (final NumberFormatException e) { - LOG.error("Unable to parse ServiceTemplateInstance ID out of serviceInstanceID: {}", serviceInstanceID); - } - } else { - LOG.error("Unable to parse ServiceTemplateInstance ID out of serviceInstanceID because it is null!"); - } - return Long.MIN_VALUE; - } + /** + * Determine the ServiceTemplateInstanceId long from the ServiceInstanceId + * QName. + * + * @return the ServiceTemplateInstanceId if the retrieval is successful, + * Long.MIN_VALUE otherwise + */ + public static long determineServiceTemplateInstanceId(final URI serviceInstanceID) { + if (Objects.nonNull(serviceInstanceID)) { + try { + if (serviceInstanceID.toString().contains("/")) { + return Long.parseLong(StringUtils.substringAfterLast(serviceInstanceID.toString(), "/")); + } else { + return Long.parseLong(serviceInstanceID.toString()); + } + } catch (final NumberFormatException e) { + LOG.error("Unable to parse ServiceTemplateInstance ID out of serviceInstanceID: {}", serviceInstanceID); + } + } else { + LOG.error("Unable to parse ServiceTemplateInstance ID out of serviceInstanceID because it is null!"); + } + return Long.MIN_VALUE; + } - /** - * Checks if a certain property was specified in the Tosca.xml of the ArtifactTemplate and returns it if so. - * - * @param artifactTemplate the ID of the ArtifactTemplate - * @param propertyName the name of the property - * @return the property value if specified, null otherwise - */ - public static String getProperty(final TArtifactTemplate artifactTemplate, final String propertyName) { - final Document properties = ToscaEngine.getEntityTemplateProperties(artifactTemplate); - // check if there are specified properties at all - if (properties == null) { - return null; - } + /** + * Checks if a certain property was specified in the Tosca.xml of the + * ArtifactTemplate and returns it if so. + * + * @param artifactTemplate the ID of the ArtifactTemplate + * @param propertyName the name of the property + * @return the property value if specified, null otherwise + */ + public static String getProperty(final TArtifactTemplate artifactTemplate, final String propertyName) { + final Document properties = ToscaEngine.getEntityTemplateProperties(artifactTemplate); + // check if there are specified properties at all + if (properties == null || !properties.hasChildNodes()) { + return null; + } - final NodeList list = properties.getFirstChild().getChildNodes(); - // iterate through properties and check name - for (int i = 0; i < list.getLength(); i++) { - final Node propNode = list.item(i); - final String localName = propNode.getLocalName(); - if (localName != null && localName.equals(propertyName)) { - return propNode.getTextContent().trim(); - } - } - return null; - } + final NodeList list = properties.getFirstChild().getChildNodes(); + // iterate through properties and check name + for (int i = 0; i < list.getLength(); i++) { + final Node propNode = list.item(i); + final String localName = propNode.getLocalName(); + if (localName != null && localName.equals(propertyName)) { + return propNode.getTextContent().trim(); + } + } + return null; + } - /** - * Checks if a PortType property was specified in the Tosca.xml of the ArtifactTemplate and returns it if so. - * - * @param artifactTemplate the ArtifactTemplate - * @return the PortType property value as QName if specified, null otherwise - */ - public static QName getPortTypeQName(final TArtifactTemplate artifactTemplate) { - try { - QName portType = QName.valueOf(getProperty(artifactTemplate, "PortType")); - LOG.debug("PortType property: {}", portType.toString()); - return portType; - } catch (final IllegalArgumentException e) { - LOG.warn("PortType property can not be parsed to QName."); - } - return null; - } + /** + * Checks if a PortType property was specified in the Tosca.xml of the + * ArtifactTemplate and returns it if so. + * + * @param artifactTemplate the ArtifactTemplate + * @return the PortType property value as QName if specified, null otherwise + */ + public static QName getPortTypeQName(final TArtifactTemplate artifactTemplate) { + try { + QName portType = QName.valueOf(getProperty(artifactTemplate, "PortType")); + LOG.debug("PortType property: {}", portType.toString()); + return portType; + } catch (final IllegalArgumentException e) { + LOG.warn("PortType property can not be parsed to QName."); + } + return null; + } }