TOMEE-2330 fix issue with incorrect base URL for generated persistence units


Project: http://git-wip-us.apache.org/repos/asf/tomee/repo
Commit: http://git-wip-us.apache.org/repos/asf/tomee/commit/224e833d
Tree: http://git-wip-us.apache.org/repos/asf/tomee/tree/224e833d
Diff: http://git-wip-us.apache.org/repos/asf/tomee/diff/224e833d

Branch: refs/heads/tomee-7.1.x
Commit: 224e833dc8e4a36fb96a8450bfd0e8571c8e845d
Parents: 1397367
Author: Jonathan Gallimore <[email protected]>
Authored: Mon Dec 10 14:16:43 2018 +0000
Committer: Otavio Santana <[email protected]>
Committed: Mon Dec 17 14:00:56 2018 -0200

----------------------------------------------------------------------
 .../main/java/org/apache/openejb/config/CmpJpaConversion.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/tomee/blob/224e833d/container/openejb-core/src/main/java/org/apache/openejb/config/CmpJpaConversion.java
----------------------------------------------------------------------
diff --git 
a/container/openejb-core/src/main/java/org/apache/openejb/config/CmpJpaConversion.java
 
b/container/openejb-core/src/main/java/org/apache/openejb/config/CmpJpaConversion.java
index ffe065a..b45de3e 100644
--- 
a/container/openejb-core/src/main/java/org/apache/openejb/config/CmpJpaConversion.java
+++ 
b/container/openejb-core/src/main/java/org/apache/openejb/config/CmpJpaConversion.java
@@ -266,10 +266,10 @@ public class CmpJpaConversion implements DynamicDeployer {
 
     private String getPersistenceModuleId(final AppModule appModule) {
         if (appModule.getModuleId() != null) {
-            return appModule.getModuleId();
+            return appModule.getJarLocation();
         }
         for (final EjbModule ejbModule : appModule.getEjbModules()) {
-            return ejbModule.getModuleId();
+            return ejbModule.getJarLocation();
         }
         throw new IllegalStateException("Comp must be in an ejb module, this 
one has none: " + appModule);
     }

Reply via email to