svn commit: r1386747 - in /openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config: DeploymentModule.java EjbJarInfoBuilder.java

[rmannibucau]

Author: rmannibucau
Date: Mon Sep 17 17:46:05 2012
New Revision: 1386747

URL: http://svn.apache.org/viewvc?rev=1386747&view=rev
Log:
TOMEE-419 handling of module-name (for jndi names)

Modified:
    
openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentModule.java
    
openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config/EjbJarInfoBuilder.java

Modified: 
openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentModule.java
URL: 
http://svn.apache.org/viewvc/openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentModule.java?rev=1386747&r1=1386746&r2=1386747&view=diff
==============================================================================
--- 
openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentModule.java
 (original)
+++ 
openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentModule.java
 Mon Sep 17 17:46:05 2012
@@ -79,10 +79,10 @@ public interface DeploymentModule {
         }
 
         private String name(NamedModule vendor, NamedModule spec, URI uri, 
File location, String name, DeploymentModule module) {
+            if (spec != null && spec.getModuleName() != null) return 
spec.getModuleName().trim(); // used to override defaults so do it first
             if (name != null && !name.startsWith("@")) return name;
             if (vendor != null && vendor.getModuleName() != null) return 
vendor.getModuleName().trim();
             if (vendor != null && vendor.getId() != null) return 
vendor.getId().trim();
-            if (spec != null && spec.getModuleName() != null) return 
spec.getModuleName().trim();
             if (spec != null && spec.getId() != null) return 
spec.getId().trim();
             if (uri != null) return stripExtension(uri.getPath());
             if (location != null && 
SystemInstance.get().getOptions().get(OPENEJB_MODULENAME_USE_HASH, false)) 
return moduleName(location) + module.hashCode();

Modified: 
openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config/EjbJarInfoBuilder.java
URL: 
http://svn.apache.org/viewvc/openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config/EjbJarInfoBuilder.java?rev=1386747&r1=1386746&r2=1386747&view=diff
==============================================================================
--- 
openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config/EjbJarInfoBuilder.java
 (original)
+++ 
openejb/branches/openejb-4.1.0/container/openejb-core/src/main/java/org/apache/openejb/config/EjbJarInfoBuilder.java
 Mon Sep 17 17:46:05 2012
@@ -141,9 +141,13 @@ public class EjbJarInfoBuilder {
 
         EjbJarInfo ejbJar = new EjbJarInfo();
         ejbJar.path = jar.getJarLocation();
-        ejbJar.moduleName = jar.getModuleId();
         ejbJar.moduleUri = jar.getModuleUri();
         ejbJar.moduleId = jar.getModuleId();
+        if (jar.getEjbJar() != null && jar.getEjbJar().getModuleName() != 
null) {
+            ejbJar.moduleName = jar.getEjbJar().getModuleName();
+        } else {
+            ejbJar.moduleName = jar.getModuleId();
+        }
 
         ejbJar.watchedResources.addAll(jar.getWatchedResources());