svn commit: r1616514 - /ofbiz/trunk/framework/catalina/src/org/ofbiz/catalina/container/CatalinaContainer.java

[jacopoc]

Author: jacopoc
Date: Thu Aug  7 15:51:43 2014
New Revision: 1616514

URL: http://svn.apache.org/r1616514
Log:
Improved the way we start Tomcat embedded to remove a series of warnings at startup.

Modified:
    ofbiz/trunk/framework/catalina/src/org/ofbiz/catalina/container/CatalinaContainer.java

Modified: ofbiz/trunk/framework/catalina/src/org/ofbiz/catalina/container/CatalinaContainer.java
URL: http://svn.apache.org/viewvc/ofbiz/trunk/framework/catalina/src/org/ofbiz/catalina/container/CatalinaContainer.java?rev=1616514&r1=1616513&r2=1616514&view=diff
==============================================================================
--- ofbiz/trunk/framework/catalina/src/org/ofbiz/catalina/container/CatalinaContainer.java (original)
+++ ofbiz/trunk/framework/catalina/src/org/ofbiz/catalina/container/CatalinaContainer.java Thu Aug  7 15:51:43 2014
@@ -237,13 +237,6 @@ public class CatalinaContainer implement
     }
 
     public boolean start() throws ContainerException {
-        // Start the Tomcat server
-        try {
-            tomcat.getServer().start();
-        } catch (LifecycleException e) {
-            throw new ContainerException(e);
-        }
-
         // load the web applications
         loadComponents();
 
@@ -260,6 +253,12 @@ public class CatalinaContainer implement
                     (con.getSecure() ? "secure" : "not-secure") + " [" + con.getProtocolHandlerClassName() + "] started.", module);
             }
         }
+        // Start the Tomcat server
+        try {
+            tomcat.getServer().start();
+        } catch (LifecycleException e) {
+            throw new ContainerException(e);
+        }
         Debug.logInfo("Started " + ServerInfo.getServerInfo(), module);
         return true;
     }
@@ -607,7 +606,6 @@ public class CatalinaContainer implement
             public Context call() throws ContainerException, LifecycleException {
                 StandardContext context = configureContext(engine, host, appInfo);
                 context.setParent(host);
-                context.start();
                 return context;
             }
         };