You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by rm...@apache.org on 2012/11/08 10:36:45 UTC

svn commit: r1406982 - /openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java

Author: rmannibucau
Date: Thu Nov  8 09:36:45 2012
New Revision: 1406982

URL: http://svn.apache.org/viewvc?rev=1406982&view=rev
Log:
formatting/typo + persistencexml list can contain a single entry

Modified:
    openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java

Modified: openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java
URL: http://svn.apache.org/viewvc/openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java?rev=1406982&r1=1406981&r2=1406982&view=diff
==============================================================================
--- openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java (original)
+++ openejb/trunk/openejb/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java Thu Nov  8 09:36:45 2012
@@ -193,10 +193,10 @@ public class DeploymentLoader implements
                 final ResourceFinder finder = new ResourceFinder("", urls.toArray(new URL[urls.size()]));
                 otherDD.putAll(getDescriptors(finder, false));
 
-                // "persistence.xml" is done separately since we mange alist of url and not s single url
+                // "persistence.xml" is done separately since we manage a list of url and not s single url
                 try {
                     final List<URL> persistenceXmls = finder.findAll(ddDir + "persistence.xml");
-                    if (persistenceXmls.size() > 1) {
+                    if (persistenceXmls.size() >= 1) {
                         final URL old = (URL) otherDD.get("persistence.xml");
                         if (old != null && !persistenceXmls.contains(old)) {
                             persistenceXmls.add(old);
@@ -623,9 +623,7 @@ public class DeploymentLoader implements
     }
 
     private WebModule createWebModule(final String jar, final URL warUrl, final ClassLoader parentClassLoader, final String contextRoot, final String moduleName) throws OpenEJBException {
-        final String warPath = URLs.toFilePath(warUrl);
-        final WebModule webModule = createWebModule(jar, warPath, parentClassLoader, contextRoot, moduleName);
-        return webModule;
+        return createWebModule(jar, URLs.toFilePath(warUrl), parentClassLoader, contextRoot, moduleName);
     }
 
     public void addWebModule(final AppModule appModule, final URL warUrl, final ClassLoader parentClassLoader, final String contextRoot, final String moduleName) throws OpenEJBException {