You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by db...@apache.org on 2013/03/20 02:17:45 UTC

svn commit: r1458601 - in /tomee/tomee/branches/tomee-1.5.2: container/openejb-core/src/main/java/org/apache/openejb/config/ examples/projectstage-demo/ examples/simple-stateless-with-descriptor/

Author: dblevins
Date: Wed Mar 20 01:17:44 2013
New Revision: 1458601

URL: http://svn.apache.org/r1458601
Log:
reverting TOMEE-727 and more finalizing versions

Modified:
    tomee/tomee/branches/tomee-1.5.2/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java
    tomee/tomee/branches/tomee-1.5.2/container/openejb-core/src/main/java/org/apache/openejb/config/OutputGeneratedDescriptors.java
    tomee/tomee/branches/tomee-1.5.2/examples/projectstage-demo/pom.xml
    tomee/tomee/branches/tomee-1.5.2/examples/simple-stateless-with-descriptor/pom.xml

Modified: tomee/tomee/branches/tomee-1.5.2/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.5.2/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java?rev=1458601&r1=1458600&r2=1458601&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.5.2/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java (original)
+++ tomee/tomee/branches/tomee-1.5.2/container/openejb-core/src/main/java/org/apache/openejb/config/DeploymentLoader.java Wed Mar 20 01:17:44 2013
@@ -31,6 +31,7 @@ import org.apache.openejb.jee.Connector;
 import org.apache.openejb.jee.EjbJar;
 import org.apache.openejb.jee.FacesConfig;
 import org.apache.openejb.jee.JavaWsdlMapping;
+import org.apache.openejb.jee.JaxbJavaee;
 import org.apache.openejb.jee.JspConfig;
 import org.apache.openejb.jee.Module;
 import org.apache.openejb.jee.Taglib;
@@ -42,7 +43,6 @@ import org.apache.openejb.jee.oejb3.Open
 import org.apache.openejb.loader.FileUtils;
 import org.apache.openejb.loader.IO;
 import org.apache.openejb.loader.SystemInstance;
-import org.apache.openejb.sxc.ApplicationXml;
 import org.apache.openejb.util.AnnotationFinder;
 import org.apache.openejb.util.JarExtractor;
 import org.apache.openejb.util.LogCategory;
@@ -54,7 +54,9 @@ import org.apache.xbean.finder.UrlSet;
 import org.apache.xbean.finder.archive.ClassesArchive;
 import org.apache.xbean.finder.filter.Filter;
 import org.apache.xbean.finder.filter.Filters;
+import org.xml.sax.SAXException;
 
+import javax.xml.bind.JAXBException;
 import java.io.BufferedInputStream;
 import java.io.File;
 import java.io.IOException;
@@ -358,7 +360,7 @@ public class DeploymentLoader implements
             final Application application;
             if (applicationXmlUrl != null) {
 
-                application = unmarshal(applicationXmlUrl);
+                application = unmarshal(Application.class, "application.xml", applicationXmlUrl);
                 for (final Module module : application.getModule()) {
                     try {
                         if (module.getEjb() != null) {
@@ -1504,11 +1506,17 @@ public class DeploymentLoader implements
     }
 
     @SuppressWarnings({"unchecked"})
-    public static Application unmarshal(final URL url) throws OpenEJBException {
+    public static <T> T unmarshal(final Class<T> type, final String descriptor, final URL url) throws OpenEJBException {
         try {
-            return ApplicationXml.unmarshal(url);
+            return (T) JaxbJavaee.unmarshalJavaee(type, new BufferedInputStream(url.openStream()));
+        } catch (SAXException e) {
+            throw new OpenEJBException("Cannot parse the " + descriptor + " file: " + url.toExternalForm(), e);
+        } catch (JAXBException e) {
+            throw new OpenEJBException("Cannot unmarshall the " + descriptor + " file: " + url.toExternalForm(), e);
+        } catch (IOException e) {
+            throw new OpenEJBException("Cannot read the " + descriptor + " file: " + url.toExternalForm(), e);
         } catch (Exception e) {
-            throw new OpenEJBException("Encountered error parsing the application.xml file: " + url.toExternalForm(), e);
+            throw new OpenEJBException("Encountered unknown error parsing the " + descriptor + " file: " + url.toExternalForm(), e);
         }
     }
 

Modified: tomee/tomee/branches/tomee-1.5.2/container/openejb-core/src/main/java/org/apache/openejb/config/OutputGeneratedDescriptors.java
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.5.2/container/openejb-core/src/main/java/org/apache/openejb/config/OutputGeneratedDescriptors.java?rev=1458601&r1=1458600&r2=1458601&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.5.2/container/openejb-core/src/main/java/org/apache/openejb/config/OutputGeneratedDescriptors.java (original)
+++ tomee/tomee/branches/tomee-1.5.2/container/openejb-core/src/main/java/org/apache/openejb/config/OutputGeneratedDescriptors.java Wed Mar 20 01:17:44 2013
@@ -32,7 +32,6 @@ import org.apache.openejb.jee.oejb3.Open
 import org.apache.openejb.loader.IO;
 import org.apache.openejb.loader.Options;
 import org.apache.openejb.loader.SystemInstance;
-import org.apache.openejb.sxc.EjbJarXml;
 import org.apache.openejb.util.LogCategory;
 import org.apache.openejb.util.Logger;
 
@@ -215,7 +214,7 @@ public class OutputGeneratedDescriptors 
 
             final OutputStream out = IO.write(tempFile);
             try {
-                EjbJarXml.marshal(ejbJar, out);
+                JaxbJavaee.marshal(EjbJar.class, ejbJar, out);
                 logger.info("Dumping Generated ejb-jar.xml to: " + tempFile.getAbsolutePath());
             } catch (JAXBException e) {
             } finally {

Modified: tomee/tomee/branches/tomee-1.5.2/examples/projectstage-demo/pom.xml
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.5.2/examples/projectstage-demo/pom.xml?rev=1458601&r1=1458600&r2=1458601&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.5.2/examples/projectstage-demo/pom.xml (original)
+++ tomee/tomee/branches/tomee-1.5.2/examples/projectstage-demo/pom.xml Wed Mar 20 01:17:44 2013
@@ -23,7 +23,7 @@
 
   <groupId>org.superbiz</groupId>
   <artifactId>projectstage-demo</artifactId>
-  <version>1.0-SNAPSHOT</version>
+  <version>1.0</version>
   <name>OpenEJB :: Examples :: DeltaSpike ProjectStage</name>
 
   <dependencies>

Modified: tomee/tomee/branches/tomee-1.5.2/examples/simple-stateless-with-descriptor/pom.xml
URL: http://svn.apache.org/viewvc/tomee/tomee/branches/tomee-1.5.2/examples/simple-stateless-with-descriptor/pom.xml?rev=1458601&r1=1458600&r2=1458601&view=diff
==============================================================================
--- tomee/tomee/branches/tomee-1.5.2/examples/simple-stateless-with-descriptor/pom.xml (original)
+++ tomee/tomee/branches/tomee-1.5.2/examples/simple-stateless-with-descriptor/pom.xml Wed Mar 20 01:17:44 2013
@@ -74,12 +74,6 @@
       <version>4.5.2</version>
       <scope>test</scope>
     </dependency>
-    <dependency>
-      <groupId>org.apache.openejb</groupId>
-      <artifactId>openejb-jee-accessors</artifactId>
-      <version>4.5.2</version>
-      <scope>test</scope>
-    </dependency>
   </dependencies>
 
   <!--