Author: rmannibucau
Date: Sun Oct 12 13:24:37 2014
New Revision: 1631174
URL: http://svn.apache.org/r1631174
Log:
avoid NPE in InstanceImpl
Modified:
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/OwbCDI.java
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/inject/instance/InstanceImpl.java
openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/container/OwbCDIProviderTest.java
Modified:
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/OwbCDI.java
URL:
http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/OwbCDI.java?rev=1631174&r1=1631173&r2=1631174&view=diff
==============================================================================
---
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/OwbCDI.java
(original)
+++
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/OwbCDI.java
Sun Oct 12 13:24:37 2014
@@ -26,17 +26,12 @@ import org.apache.webbeans.inject.instan
import javax.enterprise.inject.Instance;
import javax.enterprise.inject.spi.BeanManager;
import javax.enterprise.inject.spi.CDI;
-import javax.enterprise.inject.spi.InjectionPoint;
import javax.enterprise.util.TypeLiteral;
-import javax.inject.Inject;
import java.lang.annotation.Annotation;
import java.util.Iterator;
public class OwbCDI extends CDI<Object>
{
- @Inject
- private Instance<Object> injectionPoint;
-
private WebBeansContext getWebBeansContext()
{
return WebBeansContext.currentInstance();
@@ -47,8 +42,6 @@ public class OwbCDI extends CDI<Object>
final WebBeansContext webBeansContext = getWebBeansContext();
final BeanManagerImpl bm = webBeansContext.getBeanManagerImpl();
final CreationalContextImpl<Instance<Object>> creationalContext =
bm.createCreationalContext(null);
- final InjectionPoint injectionPoint1 =
bm.createInjectionTarget(bm.createAnnotatedType(OwbCDI.class)).getInjectionPoints().iterator().next();
- creationalContext.putInjectionPoint(injectionPoint1);
return new
InstanceBean<Object>(webBeansContext).create(creationalContext);
}
Modified:
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/inject/instance/InstanceImpl.java
URL:
http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/inject/instance/InstanceImpl.java?rev=1631174&r1=1631173&r2=1631174&view=diff
==============================================================================
---
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/inject/instance/InstanceImpl.java
(original)
+++
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/inject/instance/InstanceImpl.java
Sun Oct 12 13:24:37 2014
@@ -163,7 +163,7 @@ public class InstanceImpl<T> implements
InjectionResolver injectionResolver =
webBeansContext.getBeanManagerImpl().getInjectionResolver();
- Bean<?> injectionPointBean = injectionPoint.getBean();
+ Bean<?> injectionPointBean = injectionPoint != null ?
injectionPoint.getBean() : null;
Class<?> injectionPointClass = null;
if (injectionPointBean != null)
{
Modified:
openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/container/OwbCDIProviderTest.java
URL:
http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/container/OwbCDIProviderTest.java?rev=1631174&r1=1631173&r2=1631174&view=diff
==============================================================================
---
openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/container/OwbCDIProviderTest.java
(original)
+++
openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/container/OwbCDIProviderTest.java
Sun Oct 12 13:24:37 2014
@@ -36,6 +36,7 @@ public class OwbCDIProviderTest extends
assertNotNull(CDI.current());
assertNotNull(CDI.current().getBeanManager());
assertFalse(CDI.current().isUnsatisfied());
- assertTrue(CDI.current().isAmbiguous());
+ assertFalse(CDI.current().isAmbiguous());
+ assertNotNull(CDI.current().get());
}
}