If you have looked at 

http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/BeanManagerImpl.java?r1=915744&r2=922040&diff_format=h

you can see why I changed it a times ago!



________________________________
From: "[email protected]" <[email protected]>
To: [email protected]
Sent: Wed, March 24, 2010 8:48:01 PM
Subject: svn commit: r927148 - 
/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/BeanManagerImpl.java

Author: struberg
Date: Wed Mar 24 18:48:00 2010
New Revision: 927148

URL: http://svn.apache.org/viewvc?rev=927148&view=rev
Log:
OWB-329 only add dependent CreationalContext for @Dpendent beans

This needs to be reverted, otherwise I got all @ApplicationScoped beans directly
into the CreationalContext of another @NormalScopedBeans, which lead to 
SerializationExceptions

Modified:
    
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/BeanManagerImpl.java

Modified: 
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/BeanManagerImpl.java
URL: 
http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/BeanManagerImpl.java?rev=927148&r1=927147&r2=927148&view=diff
==============================================================================
--- 
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/BeanManagerImpl.java
 (original)
+++ 
openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/container/BeanManagerImpl.java
 Wed Mar 24 18:48:00 2010
@@ -714,9 +714,11 @@ public class BeanManagerImpl implements 
         Bean<Object> injectedBean = 
(Bean<Object>)injectionResolver.getInjectionPointBean(injectionPoint);
         CreationalContextImpl<Object> injectedCreational = 
(CreationalContextImpl<Object>)createCreationalContext(injectedBean);
        
-        injectedCreational.setOwnerCreational(ownerCreationalContextImpl);
         if(WebBeansUtil.isDependent(injectedBean))
         {        
+            // this must only be added for dependent beans, otherwise we 
register @NormalScoped beans as dependent!
+            injectedCreational.setOwnerCreational(ownerCreationalContextImpl);
+            
             //Creating a new creational context for target bean instance
             instance = getReference(injectedBean, injectionPoint.getType(), 
injectedCreational);
            
@@ -836,7 +838,7 @@ public class BeanManagerImpl implements 
             }
            
             //Get dependent from DependentContex that create contextual 
instance
-            instance = context.get((Bean<Object>)bean, 
(CreationalContext<Object>)creationalContext);    
+            instance = context.get((Bean<Object>)bean, 
(CreationalContext<Object>)creationalContext);
         }
        
         return instance;


      ___________________________________________________________________
Yahoo! Türkiye açıldı!  http://yahoo.com.tr
İnternet üzerindeki en iyi içeriği Yahoo! Türkiye sizlere sunuyor!

Reply via email to