Author: remm
Date: Thu Dec 13 15:04:32 2012
New Revision: 1421327

URL: http://svn.apache.org/viewvc?rev=1421327&view=rev
Log:
Try out more accurate comparison with the connector bean and the protocol 
handler (the default constructor does not create the same protocol handler).

Modified:
    
tomcat/sandbox/storeconfig6/trunk/src/main/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java
    
tomcat/sandbox/storeconfig7/trunk/src/main/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java

Modified: 
tomcat/sandbox/storeconfig6/trunk/src/main/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java
URL: 
http://svn.apache.org/viewvc/tomcat/sandbox/storeconfig6/trunk/src/main/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java?rev=1421327&r1=1421326&r2=1421327&view=diff
==============================================================================
--- 
tomcat/sandbox/storeconfig6/trunk/src/main/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java
 (original)
+++ 
tomcat/sandbox/storeconfig6/trunk/src/main/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java
 Thu Dec 13 15:04:32 2012
@@ -87,11 +87,13 @@ public class ConnectorStoreAppender exte
             writer.print("\"");
         }
 
-        List propertyKeys = getPropertyKeys((Connector) bean);
+        Connector connector = (Connector) bean;
+        String protocol = connector.getProtocol();
+        List propertyKeys = getPropertyKeys(connector);
         // Create blank instance
-        Object bean2 = defaultInstance(bean);
-        for (Iterator propertyIterator = propertyKeys.iterator(); 
propertyIterator
-                .hasNext();) {
+        Object bean2 = new Connector(protocol);//defaultInstance(bean);
+        Iterator propertyIterator = propertyKeys.iterator();
+        while (propertyIterator.hasNext()) {
             String key = (String) propertyIterator.next();
             Object value = (Object) IntrospectionUtils.getProperty(bean, key);
 
@@ -113,10 +115,10 @@ public class ConnectorStoreAppender exte
                 // The property has its default value
                 continue;
             }
-            if (isPrintValue(bean, bean2, key, desc))
+            if (isPrintValue(bean, bean2, key, desc)) {
                 printValue(writer, indent, key, value);
+            }
         }
-        String protocol = ((Connector) bean).getProtocol();
         if (protocol != null && !"HTTP/1.1".equals(protocol))
             super.printValue(writer, indent, "protocol", protocol);
 

Modified: 
tomcat/sandbox/storeconfig7/trunk/src/main/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java
URL: 
http://svn.apache.org/viewvc/tomcat/sandbox/storeconfig7/trunk/src/main/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java?rev=1421327&r1=1421326&r2=1421327&view=diff
==============================================================================
--- 
tomcat/sandbox/storeconfig7/trunk/src/main/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java
 (original)
+++ 
tomcat/sandbox/storeconfig7/trunk/src/main/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java
 Thu Dec 13 15:04:32 2012
@@ -87,9 +87,11 @@ public class ConnectorStoreAppender exte
             writer.print("\"");
         }
 
-        List propertyKeys = getPropertyKeys((Connector) bean);
+        Connector connector = (Connector) bean;
+        String protocol = connector.getProtocol();
+        List propertyKeys = getPropertyKeys(connector);
         // Create blank instance
-        Object bean2 = defaultInstance(bean);
+        Object bean2 = new Connector(protocol);//defaultInstance(bean);
         Iterator propertyIterator = propertyKeys.iterator();
         while (propertyIterator.hasNext()) {
             String key = (String) propertyIterator.next();
@@ -117,7 +119,6 @@ public class ConnectorStoreAppender exte
                 printValue(writer, indent, key, value);
             }
         }
-        String protocol = ((Connector) bean).getProtocol();
         if (protocol != null && !"HTTP/1.1".equals(protocol))
             super.printValue(writer, indent, "protocol", protocol);
 



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to