Author: remm
Date: Thu Oct  3 16:23:59 2013
New Revision: 1528920

URL: http://svn.apache.org/r1528920
Log:
- Fix some classnames for resources following refactorings
- Add support for InetAddress for the connector address
- Do not ignore the thread local listener and add the special engine listener 
just in case

Modified:
    
tomcat/trunk/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java
    
tomcat/trunk/java/org/apache/catalina/storeconfig/GlobalNamingResourcesSF.java
    tomcat/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java
    tomcat/trunk/java/org/apache/catalina/storeconfig/server-registry.xml

Modified: 
tomcat/trunk/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java?rev=1528920&r1=1528919&r2=1528920&view=diff
==============================================================================
--- 
tomcat/trunk/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java 
(original)
+++ 
tomcat/trunk/java/org/apache/catalina/storeconfig/ConnectorStoreAppender.java 
Thu Oct  3 16:23:59 2013
@@ -23,7 +23,6 @@ import java.beans.PropertyDescriptor;
 import java.io.File;
 import java.io.IOException;
 import java.io.PrintWriter;
-import java.net.InetAddress;
 import java.util.ArrayList;
 import java.util.HashMap;
 import java.util.Iterator;
@@ -101,10 +100,6 @@ public class ConnectorStoreAppender exte
             if (value == null) {
                 continue; // Null values are not persisted
             }
-            // Convert IP addresses to strings so they will be persisted
-            if (value instanceof InetAddress) {
-                value = ((InetAddress) value).getHostAddress();
-            }
             if (!isPersistable(value.getClass())) {
                 continue;
             }

Modified: 
tomcat/trunk/java/org/apache/catalina/storeconfig/GlobalNamingResourcesSF.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/storeconfig/GlobalNamingResourcesSF.java?rev=1528920&r1=1528919&r2=1528920&view=diff
==============================================================================
--- 
tomcat/trunk/java/org/apache/catalina/storeconfig/GlobalNamingResourcesSF.java 
(original)
+++ 
tomcat/trunk/java/org/apache/catalina/storeconfig/GlobalNamingResourcesSF.java 
Thu Oct  3 16:23:59 2013
@@ -56,7 +56,7 @@ public class GlobalNamingResourcesSF ext
                             resources);
                 } else {
                     if(log.isWarnEnabled())
-                        log.warn("Can't find NamingRsources Store Factory!");
+                        log.warn("Can't find NamingResources Store Factory!");
                 }
 
                 getStoreAppender().printIndent(aWriter, indent + 2);

Modified: tomcat/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java?rev=1528920&r1=1528919&r2=1528920&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/storeconfig/StoreAppender.java Thu 
Oct  3 16:23:59 2013
@@ -21,6 +21,7 @@ import java.beans.IndexedPropertyDescrip
 import java.beans.Introspector;
 import java.beans.PropertyDescriptor;
 import java.io.PrintWriter;
+import java.net.InetAddress;
 import java.util.Iterator;
 
 import org.apache.tomcat.util.IntrospectionUtils;
@@ -38,7 +39,7 @@ public class StoreAppender {
             Integer.TYPE, Boolean.class, Boolean.TYPE, Byte.class, Byte.TYPE,
             Character.class, Character.TYPE, Double.class, Double.TYPE,
             Float.class, Float.TYPE, Long.class, Long.TYPE, Short.class,
-            Short.TYPE, };
+            Short.TYPE, InetAddress.class };
 
     /**
      * print the closing tag
@@ -328,6 +329,10 @@ public class StoreAppender {
      */
     public void printValue(PrintWriter writer, int indent, String name,
             Object value) {
+        // Convert IP addresses to strings so they will be persisted
+        if (value instanceof InetAddress) {
+            value = ((InetAddress) value).getHostAddress();
+        }
         if (!(value instanceof String)) {
             value = value.toString();
         }
@@ -377,7 +382,7 @@ public class StoreAppender {
     protected boolean isPersistable(Class<?> clazz) {
 
         for (int i = 0; i < persistables.length; i++) {
-            if (persistables[i] == clazz) {
+            if (persistables[i] == clazz || 
persistables[i].isAssignableFrom(clazz)) {
                 return (true);
             }
         }

Modified: tomcat/trunk/java/org/apache/catalina/storeconfig/server-registry.xml
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/storeconfig/server-registry.xml?rev=1528920&r1=1528919&r2=1528920&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/catalina/storeconfig/server-registry.xml 
(original)
+++ tomcat/trunk/java/org/apache/catalina/storeconfig/server-registry.xml Thu 
Oct  3 16:23:59 2013
@@ -82,13 +82,13 @@
         <TransientAttribute>ignoreAnnotations</TransientAttribute>
      </Description>
      <Description
-        id="org.apache.catalina.deploy.NamingResources.[GlobalNamingResources]"
+        
id="org.apache.catalina.deploy.NamingResourcesImpl.[GlobalNamingResources]"
         tag="GlobalNamingResources"
         standard="true"
         default="false"
         attributes="false"
         children="true"
-        tagClass="org.apache.catalina.deploy.NamingResources"
+        tagClass="org.apache.catalina.deploy.NamingResourcesImpl"
         
storeFactoryClass="org.apache.catalina.storeconfig.GlobalNamingResourcesSF">
      </Description>
       <Description
@@ -109,7 +109,7 @@
         default="false"
         attributes="false"
         children="true"
-        tagClass="org.apache.catalina.deploy.NamingResources"
+        tagClass="org.apache.catalina.deploy.NamingResourcesImpl"
         storeFactoryClass="org.apache.catalina.storeconfig.NamingResourcesSF">
      </Description>
      <Description
@@ -168,7 +168,7 @@
         tag="Parameter"
         standard="true"
         default="false"
-        tagClass="org.apache.catalina.deploy.ApplicationParameter"
+        tagClass="org.apache.tomcat.util.descriptor.web.ApplicationParameter"
         storeFactoryClass="org.apache.catalina.storeconfig.StoreFactoryBase">
      </Description>
      <Description
@@ -191,7 +191,7 @@
        <TransientChild>org.apache.catalina.startup.HostConfig</TransientChild>
        
<TransientChild>org.apache.catalina.core.StandardHost$MemoryLeakTrackingListener</TransientChild>
        
<TransientChild>org.apache.catalina.mapper.MapperListener</TransientChild>
-       
<TransientChild>org.apache.catalina.core.ThreadLocalLeakPreventionListener</TransientChild>
+       
<TransientChild>org.apache.catalina.core.StandardEngine$AccessLogListener</TransientChild>
      </Description>
      <Description
         id="org.apache.catalina.core.StandardServer.[ServerLifecycleListener]"
@@ -223,28 +223,28 @@
         tag="Environment"
         standard="true"
         default="false"
-        tagClass="org.apache.catalina.deploy.ContextEnvironment"
+        tagClass="org.apache.tomcat.util.descriptor.web.ContextEnvironment"
         storeFactoryClass="org.apache.catalina.storeconfig.StoreFactoryBase">
      </Description>
      <Description
         tag="EJB"
         standard="true"
         default="false"
-        tagClass="org.apache.catalina.deploy.ContextEjb"
+        tagClass="org.apache.tomcat.util.descriptor.web.ContextEjb"
         storeFactoryClass="org.apache.catalina.storeconfig.StoreFactoryBase">
      </Description>
      <Description
         tag="LocalEjb"
         standard="true"
         default="false"
-        tagClass="org.apache.catalina.deploy.ContextLocalEjb"
+        tagClass="org.apache.tomcat.util.descriptor.web.ContextLocalEjb"
         storeFactoryClass="org.apache.catalina.storeconfig.StoreFactoryBase">
      </Description>
      <Description
         tag="Resource"
         standard="true"
         default="false"
-        tagClass="org.apache.catalina.deploy.ContextResource"
+        tagClass="org.apache.tomcat.util.descriptor.web.ContextResource"
         storeFactoryClass="org.apache.catalina.storeconfig.StoreFactoryBase">
      </Description>
     <Description
@@ -291,21 +291,21 @@
         tag="ResourceEnvRef"
         standard="true"
         default="false"
-        tagClass="org.apache.catalina.deploy.ContextResourceEnvRef"
+        tagClass="org.apache.tomcat.util.descriptor.web.ContextResourceEnvRef"
         storeFactoryClass="org.apache.catalina.storeconfig.StoreFactoryBase">
      </Description>
      <Description
         tag="ResourceLink"
         standard="true"
         default="false"
-        tagClass="org.apache.catalina.deploy.ContextResourceLink"
+        tagClass="org.apache.tomcat.util.descriptor.web.ContextResourceLink"
         storeFactoryClass="org.apache.catalina.storeconfig.StoreFactoryBase">
      </Description>
      <Description
         tag="Transaction"
         standard="true"
         default="false"
-        tagClass="org.apache.catalina.deploy.ContextTransaction"
+        tagClass="org.apache.tomcat.util.descriptor.web.ContextTransaction"
         storeFactoryClass="org.apache.catalina.storeconfig.StoreFactoryBase">
      </Description>
      <Description



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

Reply via email to