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 2013/04/26 08:54:58 UTC

svn commit: r1476073 - in /tomee/tomee/trunk: container/openejb-core/src/main/java/org/apache/openejb/ examples/spring-data-proxy/src/main/resources/META-INF/ examples/spring-data-proxy/src/test/resources/ examples/spring-data-proxy/src/test/resources/...

Author: rmannibucau
Date: Fri Apr 26 06:54:58 2013
New Revision: 1476073

URL: http://svn.apache.org/r1476073
Log:
spring-data sample was using cdi without activating it + fixing an issue when creating a dynamic ejb

Added:
    tomee/tomee/trunk/examples/spring-data-proxy/src/main/resources/META-INF/beans.xml
    tomee/tomee/trunk/examples/spring-data-proxy/src/test/resources/
    tomee/tomee/trunk/examples/spring-data-proxy/src/test/resources/META-INF/
    tomee/tomee/trunk/examples/spring-data-proxy/src/test/resources/META-INF/beans.xml
Modified:
    tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/BeanContext.java

Modified: tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/BeanContext.java
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/BeanContext.java?rev=1476073&r1=1476072&r2=1476073&view=diff
==============================================================================
--- tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/BeanContext.java (original)
+++ tomee/tomee/trunk/container/openejb-core/src/main/java/org/apache/openejb/BeanContext.java Fri Apr 26 06:54:58 2013
@@ -1411,9 +1411,11 @@ public class BeanContext extends Deploym
         final ThreadContext callContext = new ThreadContext(this, null, Operation.INJECTION);
         final ThreadContext oldContext = ThreadContext.enter(callContext);
 
+        final boolean dynamicallyImplemented = isDynamicallyImplemented();
+
         final WebBeansContext webBeansContext = getModuleContext().getAppContext().getWebBeansContext();
 
-        if (isDynamicallyImplemented()) {
+        if (dynamicallyImplemented) {
             if (!InvocationHandler.class.isAssignableFrom(getProxyClass())) {
                 throw new OpenEJBException("proxy class can only be InvocationHandler");
             }
@@ -1437,7 +1439,7 @@ public class BeanContext extends Deploym
             }
 
             Object rootInstance;
-            if (cdiEjbBean != null && CdiEjbBean.EjbInjectionTargetImpl.class.isInstance(cdiEjbBean.getInjectionTarget())) {
+            if (cdiEjbBean != null && !dynamicallyImplemented && CdiEjbBean.EjbInjectionTargetImpl.class.isInstance(cdiEjbBean.getInjectionTarget())) {
                 rootInstance = CdiEjbBean.EjbInjectionTargetImpl.class.cast(cdiEjbBean.getInjectionTarget()).createNewPojo(creationalContext);
             } else { // not a cdi bean
                 rootInstance = getManagedClass().newInstance();
@@ -1447,7 +1449,7 @@ public class BeanContext extends Deploym
             Object beanInstance;
 
             final InjectionProcessor injectionProcessor;
-            if (!isDynamicallyImplemented()) {
+            if (!dynamicallyImplemented) {
                 injectionProcessor = new InjectionProcessor(rootInstance, getInjections(), InjectionProcessor.unwrap(ctx));
                 beanInstance = injectionProcessor.createInstance();
                 inject(beanInstance, creationalContext);

Added: tomee/tomee/trunk/examples/spring-data-proxy/src/main/resources/META-INF/beans.xml
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/examples/spring-data-proxy/src/main/resources/META-INF/beans.xml?rev=1476073&view=auto
==============================================================================
--- tomee/tomee/trunk/examples/spring-data-proxy/src/main/resources/META-INF/beans.xml (added)
+++ tomee/tomee/trunk/examples/spring-data-proxy/src/main/resources/META-INF/beans.xml Fri Apr 26 06:54:58 2013
@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+    Licensed to the Apache Software Foundation (ASF) under one
+    or more contributor license agreements.  See the NOTICE file
+    distributed with this work for additional information
+    regarding copyright ownership.  The ASF licenses this file
+    to you under the Apache License, Version 2.0 (the
+    "License"); you may not use this file except in compliance
+    with the License.  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+    Unless required by applicable law or agreed to in writing,
+    software distributed under the License is distributed on an
+    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+    KIND, either express or implied.  See the License for the
+    specific language governing permissions and limitations
+    under the License.
+-->
+<beans xmlns="http://java.sun.com/xml/ns/javaee"
+       xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+       xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/beans_1_0.xsd" />

Added: tomee/tomee/trunk/examples/spring-data-proxy/src/test/resources/META-INF/beans.xml
URL: http://svn.apache.org/viewvc/tomee/tomee/trunk/examples/spring-data-proxy/src/test/resources/META-INF/beans.xml?rev=1476073&view=auto
==============================================================================
--- tomee/tomee/trunk/examples/spring-data-proxy/src/test/resources/META-INF/beans.xml (added)
+++ tomee/tomee/trunk/examples/spring-data-proxy/src/test/resources/META-INF/beans.xml Fri Apr 26 06:54:58 2013
@@ -0,0 +1,22 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!--
+    Licensed to the Apache Software Foundation (ASF) under one
+    or more contributor license agreements.  See the NOTICE file
+    distributed with this work for additional information
+    regarding copyright ownership.  The ASF licenses this file
+    to you under the Apache License, Version 2.0 (the
+    "License"); you may not use this file except in compliance
+    with the License.  You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+    Unless required by applicable law or agreed to in writing,
+    software distributed under the License is distributed on an
+    "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+    KIND, either express or implied.  See the License for the
+    specific language governing permissions and limitations
+    under the License.
+-->
+<beans xmlns="http://java.sun.com/xml/ns/javaee"
+       xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+       xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/beans_1_0.xsd" />