You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@servicemix.apache.org by gn...@apache.org on 2007/02/13 11:44:58 UTC

svn commit: r506965 - in /incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi: GenerateServiceAssemblyDescriptorMojo.java XmlDescriptorHelper.java

Author: gnodet
Date: Tue Feb 13 02:44:58 2007
New Revision: 506965

URL: http://svn.apache.org/viewvc?view=rev&rev=506965
Log:
SM-830: Replace System.out printing with logger

Modified:
    incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/GenerateServiceAssemblyDescriptorMojo.java
    incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/XmlDescriptorHelper.java

Modified: incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/GenerateServiceAssemblyDescriptorMojo.java
URL: http://svn.apache.org/viewvc/incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/GenerateServiceAssemblyDescriptorMojo.java?view=diff&rev=506965&r1=506964&r2=506965
==============================================================================
--- incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/GenerateServiceAssemblyDescriptorMojo.java (original)
+++ incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/GenerateServiceAssemblyDescriptorMojo.java Tue Feb 13 02:44:58 2007
@@ -350,7 +350,7 @@
 						.next();
 				if (dependency.getArtifactId()
 						.equals(serviceUnitInfo.getName())) {
-					System.out.println("Adding "
+					getLog().debug("Adding "
 							+ serviceUnitInfo.getFilename());
 					orderedServiceUnits.add(serviceUnitInfo);
 				}

Modified: incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/XmlDescriptorHelper.java
URL: http://svn.apache.org/viewvc/incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/XmlDescriptorHelper.java?view=diff&rev=506965&r1=506964&r2=506965
==============================================================================
--- incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/XmlDescriptorHelper.java (original)
+++ incubator/servicemix/branches/servicemix-3.1/tooling/jbi-maven-plugin/src/main/java/org/apache/servicemix/maven/plugin/jbi/XmlDescriptorHelper.java Tue Feb 13 02:44:58 2007
@@ -1,85 +1,90 @@
-/*
- * Licensed to the Apache Software Foundation (ASF) under one or more
- * contributor license agreements.  See the NOTICE file distributed with
- * this work for additional information regarding copyright ownership.
- * The ASF licenses this file to You 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.
- */
-package org.apache.servicemix.maven.plugin.jbi;
-
-import javax.xml.namespace.QName;
-
-import org.w3c.dom.Element;
-import org.w3c.dom.Node;
-
-public class XmlDescriptorHelper {
-
-	public static QName getServiceName(Element childElement) {
-		if (childElement.hasAttribute("service-name")) {
-			String prefixAndLocalPart = childElement
-					.getAttribute("service-name");
-			String prefix = prefixAndLocalPart.substring(0, prefixAndLocalPart
-					.indexOf(':'));
-			String localPart = prefixAndLocalPart.substring(prefixAndLocalPart
-					.indexOf(':') + 1);
-			return new QName(childElement.lookupNamespaceURI(prefix), localPart);
-		}
-		return null;
-	}
-
-	public static QName getInterfaceName(Element childElement) {
-		if (childElement.hasAttribute("interface-name")) {
-			String prefixAndLocalPart = childElement
-					.getAttribute("interface-name");
-			String prefix = prefixAndLocalPart.substring(0, prefixAndLocalPart
-					.indexOf(':'));
-			String localPart = prefixAndLocalPart.substring(prefixAndLocalPart
-					.indexOf(':') + 1);
-			return new QName(childElement.lookupNamespaceURI(prefix), localPart);
-		}
-		return null;
-	}
-
-	public static boolean isElement(Node node, String namespaceUrl,
-			String localPart) {
-		if (node instanceof Element) {
-			Element element = (Element) node;
-			System.out.println("Got Element nodeName:" + element.getNodeName()
-					+ " namespaceUri:" + element.getNamespaceURI()
-					+ " localName:" + element.getLocalName());
-			if (localPart.equals(element.getNodeName()))
-				return true;
-			else {
-				// Compare the namespace URI and localname
-				System.out.println(namespaceUrl + "="
-						+ element.getNamespaceURI() + " is "
-						+ namespaceUrl.equals(element.getNamespaceURI()));
-				System.out.println(localPart + "="
-						+ element.getLocalName() + " is "
-						+ localPart.equals(element.getLocalName()));
-				if ((namespaceUrl.equals(element.getNamespaceURI()))
-						&& (localPart.equals(element.getLocalName()))) {
-					return true;
-				}
-			}
-		}
-		return false;
-	}
-
-	public static String getEndpointName(Element childElement) {
-		if (childElement.hasAttribute("endpoint-name")) {
-			return childElement.getAttribute("endpoint-name");
-		}
-		return null;
-	}
-
-}
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements.  See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You 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.
+ */
+package org.apache.servicemix.maven.plugin.jbi;
+
+import javax.xml.namespace.QName;
+
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
+import org.w3c.dom.Element;
+import org.w3c.dom.Node;
+
+public class XmlDescriptorHelper {
+
+        private static final Log log = LogFactory.getLog(XmlDescriptorHelper.class);
+
+	public static QName getServiceName(Element childElement) {
+		if (childElement.hasAttribute("service-name")) {
+			String prefixAndLocalPart = childElement
+					.getAttribute("service-name");
+			String prefix = prefixAndLocalPart.substring(0, prefixAndLocalPart
+					.indexOf(':'));
+			String localPart = prefixAndLocalPart.substring(prefixAndLocalPart
+					.indexOf(':') + 1);
+			return new QName(childElement.lookupNamespaceURI(prefix), localPart);
+		}
+		return null;
+	}
+
+	public static QName getInterfaceName(Element childElement) {
+		if (childElement.hasAttribute("interface-name")) {
+			String prefixAndLocalPart = childElement
+					.getAttribute("interface-name");
+			String prefix = prefixAndLocalPart.substring(0, prefixAndLocalPart
+					.indexOf(':'));
+			String localPart = prefixAndLocalPart.substring(prefixAndLocalPart
+					.indexOf(':') + 1);
+			return new QName(childElement.lookupNamespaceURI(prefix), localPart);
+		}
+		return null;
+	}
+
+	public static boolean isElement(Node node, String namespaceUrl,
+			String localPart) {
+		if (node instanceof Element) {
+			Element element = (Element) node;
+			log.debug("Got Element nodeName:" + element.getNodeName()
+					+ " namespaceUri:" + element.getNamespaceURI()
+					+ " localName:" + element.getLocalName());
+			if (localPart.equals(element.getNodeName()))
+				return true;
+			else {
+				// Compare the namespace URI and localname
+				log.debug(namespaceUrl + "="
+						+ element.getNamespaceURI() + " is "
+						+ namespaceUrl.equals(element.getNamespaceURI()));
+				log.debug(localPart + "="
+						+ element.getLocalName() + " is "
+						+ localPart.equals(element.getLocalName()));
+				if ((namespaceUrl.equals(element.getNamespaceURI()))
+						&& (localPart.equals(element.getLocalName()))) {
+					return true;
+				}
+			}
+		}
+		return false;
+	}
+
+	public static String getEndpointName(Element childElement) {
+		if (childElement.hasAttribute("endpoint-name")) {
+			return childElement.getAttribute("endpoint-name");
+		}
+		return null;
+	}
+
+}