[jboss-svn-commits] JBL Code SVN: r34458 - labs/jbossesb/trunk/product/services/soap/src/main/java/org/jboss/soa/esb/actions/soap.

[jboss-svn-commits at lists.jboss.org]

Author: mageshbk at jboss.com
Date: 2010-08-02 08:31:02 -0400 (Mon, 02 Aug 2010)
New Revision: 34458

Modified:
   labs/jbossesb/trunk/product/services/soap/src/main/java/org/jboss/soa/esb/actions/soap/JBossWSDLReader.java
Log:
[JBESB-3413] - Updated SOPs to log.trace.

Modified: labs/jbossesb/trunk/product/services/soap/src/main/java/org/jboss/soa/esb/actions/soap/JBossWSDLReader.java
===================================================================
--- labs/jbossesb/trunk/product/services/soap/src/main/java/org/jboss/soa/esb/actions/soap/JBossWSDLReader.java	2010-08-02 12:18:04 UTC (rev 34457)
+++ labs/jbossesb/trunk/product/services/soap/src/main/java/org/jboss/soa/esb/actions/soap/JBossWSDLReader.java	2010-08-02 12:31:02 UTC (rev 34458)
@@ -52,6 +52,8 @@
 import java.net.URL;
 import java.util.*;
 
+import org.jboss.logging.Logger;
+
 /**
  * A WSDLReader fork of the original wsdl4j 1.6.2 package
  * that delegates to the JBossWSEntityResolver  in
@@ -62,6 +64,7 @@
  */
 public class JBossWSDLReader implements WSDLReader
 {
+	private final static Logger log = Logger.getLogger(JBossWSDLReader.class);
 	// Used for determining the style of operations.
 	private static final List STYLE_ONE_WAY =
 			Arrays.asList(new String[]{Constants.ELEM_INPUT});
@@ -72,7 +75,6 @@
 	private static final List STYLE_NOTIFICATION =
 			Arrays.asList(new String[]{Constants.ELEM_OUTPUT});
 
-	protected boolean verbose = true;
 	protected boolean importDocuments = true;
 	protected ExtensionRegistry extReg = null;
 	protected String factoryImplName = null;
@@ -96,11 +98,6 @@
 	 *     <th>Default Value</th>
 	 *   </tr>
 	 *   <tr>
-	 *     <td><center>javax.wsdl.verbose</center></td>
-	 *     <td>If set to true, status messages will be displayed.</td>
-	 *     <td><center>true</center></td>
-	 *   </tr>
-	 *   <tr>
 	 *     <td><center>javax.wsdl.importDocuments</center></td>
 	 *     <td>If set to true, imported WSDL documents will be
 	 *         retrieved and processed.</td>
@@ -127,12 +124,8 @@
 			throw new IllegalArgumentException("Feature name must not be null.");
 		}
 
-		if (name.equals(Constants.FEATURE_VERBOSE))
+		if (name.equals(Constants.FEATURE_IMPORT_DOCUMENTS))
 		{
-			verbose = value;
-		}
-		else if (name.equals(Constants.FEATURE_IMPORT_DOCUMENTS))
-		{
 			importDocuments = value;
 		}
 		else
@@ -157,12 +150,8 @@
 			throw new IllegalArgumentException("Feature name must not be null.");
 		}
 
-		if (name.equals(Constants.FEATURE_VERBOSE))
+		if (name.equals(Constants.FEATURE_IMPORT_DOCUMENTS))
 		{
-			return verbose;
-		}
-		else if (name.equals(Constants.FEATURE_IMPORT_DOCUMENTS))
-		{
 			return importDocuments;
 		}
 		else
@@ -239,7 +228,7 @@
 			this.factory = null;
 
 			this.factoryImplName = factoryImplName;
-			//if(verbose) System.out.println("WSDLFactory Impl Name set to : "+factoryImplName);
+			log.trace("WSDLFactory Impl Name set to : "+factoryImplName);
 		}
 	}
 
@@ -462,14 +451,11 @@
 							if (QNameUtils.matches(Constants.Q_ELEM_DEFINITIONS,
 									documentElement))
 							{
-								if (verbose)
-								{
-									System.out.println("Retrieving document at '" + locationURI +
-											"'" +
-											(contextURI == null
-													? "."
-													: ", relative to '" + contextURI + "'."));
-								}
+								log.trace("Retrieving document at '" + locationURI +
+										"'" +
+										(contextURI == null
+												? "."
+												: ", relative to '" + contextURI + "'."));
 
 								String urlString =
 										(loc != null)
@@ -488,14 +474,11 @@
 
 								if (SchemaConstants.XSD_QNAME_LIST.contains(docElementQName))
 								{
-									if (verbose)
-									{
-										System.out.println("Retrieving schema wsdl:imported from '" + locationURI +
-												"'" +
-												(contextURI == null
-														? "."
-														: ", relative to '" + contextURI + "'."));
-									}
+									log.trace("Retrieving schema wsdl:imported from '" + locationURI +
+											"'" +
+											(contextURI == null
+													? "."
+													: ", relative to '" + contextURI + "'."));
 
 									WSDLFactory factory = getWSDLFactory();
 
@@ -752,15 +735,12 @@
 						continue;
 					}
 
-					if (verbose)
-					{
-						System.out.println("Retrieving schema at '" +
-								schemaRef.getSchemaLocationURI() +
-								(schema.getDocumentBaseURI() == null
-										? "'."
-										: "', relative to '" +
-										schema.getDocumentBaseURI() + "'."));
-					}
+					log.trace("Retrieving schema at '" +
+							schemaRef.getSchemaLocationURI() +
+							(schema.getDocumentBaseURI() == null
+									? "'."
+									: "', relative to '" +
+									schema.getDocumentBaseURI() + "'."));
 
 
 					InputStream inputStream = null;
@@ -2196,13 +2176,10 @@
 	{
 		try
 		{
-			if (verbose)
-			{
-				System.out.println("Retrieving document at '" + wsdlURI + "'" +
-						(contextURI == null
-								? "."
-								: ", relative to '" + contextURI + "'."));
-			}
+			log.trace("Retrieving document at '" + wsdlURI + "'" +
+					(contextURI == null
+							? "."
+							: ", relative to '" + contextURI + "'."));
 
 			URL contextURL = (contextURI != null)
 					? StringUtils.getURL(null, contextURI)
@@ -2353,10 +2330,7 @@
 
 		this.loc = locator;
 
-		if (verbose)
-		{
-			System.out.println("Retrieving document at '" + base + "'.");
-		}
+		log.trace("Retrieving document at '" + base + "'.");
 
 		try
 		{