Yan:: svn commit: r1049215 - in /openwebbeans/trunk/webbeans-impl/src: main/java/org/apache/webbeans/event/NotificationManager.java test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java

[Gurkan Erdogdu]

As pete mail, javadoc is correct therefore i reverted change before. If javadoc 
is correct those tck tests must be excluded. our code is correct because it is 
aligned with published javadoc

Sal, 14 Ara 2010 20:51 EET tarihinde djen...@apache.org şöyle yazdı:

>Author: djencks
>Date: Tue Dec 14 18:51:54 2010
>New Revision: 1049215
>
>URL: http://svn.apache.org/viewvc?rev=1049215&view=rev
>Log:
>OWB-490, CDITCK-174 reapply rev 1034955 to comply with cdi tck 1.0.2.SP1
>
>Modified:
>    
> openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java
>    
> openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java
>
>Modified: 
>openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java
>URL: 
>http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java?rev=1049215&r1=1049214&r2=1049215&view=diff
>==============================================================================
>--- 
>openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java
> (original)
>+++ 
>openwebbeans/trunk/webbeans-impl/src/main/java/org/apache/webbeans/event/NotificationManager.java
> Tue Dec 14 18:51:54 2010
>@@ -354,13 +354,7 @@ public final class NotificationManager
>         {
>             Type beanClassArg = actualArgs[0];
>             Type returnClassArg = actualArgs[1];
>-            
>-            if(isObserverMethod)
>-            {
>-                beanClassArg = actualArgs[1];
>-                returnClassArg = actualArgs[0];
>-            }
>-            
>+                        
>             if(checkEventTypeParameterForExtensions(beanClass, beanClassArg) 
> && 
>                     
> checkEventTypeParameterForExtensions(producerOrObserverReturnClass, 
> returnClassArg))
>             {
>
>Modified: 
>openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java
>URL: 
>http://svn.apache.org/viewvc/openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java?rev=1049215&r1=1049214&r2=1049215&view=diff
>==============================================================================
>--- 
>openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java
> (original)
>+++ 
>openwebbeans/trunk/webbeans-impl/src/test/java/org/apache/webbeans/newtests/portable/events/extensions/AddObserverMethodExtension.java
> Tue Dec 14 18:51:54 2010
>@@ -40,9 +40,9 @@ public class AddObserverMethodExtension 
>     
>     public static class MyBeanExtension implements Extension
>     {
>-        public static ObserverMethod<MyEvent> observerMethod;
>+        public static ObserverMethod<AddObserverMethodExtension> 
>observerMethod;
>         
>-        public void observer(@Observes ProcessObserverMethod<MyEvent, 
>AddObserverMethodExtension> event)
>+        public void observer(@Observes 
>ProcessObserverMethod<AddObserverMethodExtension, MyEvent> event)
>         {
>             observerMethod = event.getObserverMethod();
>             
>
>