[jboss-cvs] JBossAS SVN: r82383 - branches/JBPAPP_4_2_0_GA_CP/tomcat/src/main/org/jboss/web/tomcat/service/session/notification.

[jboss-cvs-commits at lists.jboss.org]

Author: jiwils
Date: 2008-12-18 13:59:52 -0500 (Thu, 18 Dec 2008)
New Revision: 82383

Modified:
   branches/JBPAPP_4_2_0_GA_CP/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/IgnoreUndeployLegacyClusteredSessionNotificationPolicy.java
Log:
Fix for JBPAPP-1473.  isHttpSessionListenerInvocationAllowed now correctly calls the super class method.

Modified: branches/JBPAPP_4_2_0_GA_CP/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/IgnoreUndeployLegacyClusteredSessionNotificationPolicy.java
===================================================================
--- branches/JBPAPP_4_2_0_GA_CP/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/IgnoreUndeployLegacyClusteredSessionNotificationPolicy.java	2008-12-18 18:58:59 UTC (rev 82382)
+++ branches/JBPAPP_4_2_0_GA_CP/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/IgnoreUndeployLegacyClusteredSessionNotificationPolicy.java	2008-12-18 18:59:52 UTC (rev 82383)
@@ -25,16 +25,16 @@
 /**
  * Does not allow invocation of HttpSessionListener or HttpSessionAttributeListener
  * during session expiration due to undeploy.
- * 
+ *
  * @author Brian Stansberry
  */
-public class IgnoreUndeployLegacyClusteredSessionNotificationPolicy 
+public class IgnoreUndeployLegacyClusteredSessionNotificationPolicy
    extends LegacyClusteredSessionNotificationPolicy
 {
    /**
     * Overrides superclass to return <code>false</code> if the cause of the
     * notification is {@link ClusteredSessionNotificationCause.UNDEPLOY}.
-    *  
+    *
     * @return <code>true</code> if <code>status.isLocallyUsed()</code>
     *         is <code>true</code> and the cause of the notification is not
     *         {@link ClusteredSessionNotificationCause.UNDEPLOY}.
@@ -42,14 +42,14 @@
    public boolean isHttpSessionAttributeListenerInvocationAllowed(ClusteredSessionManagementStatus status,
          ClusteredSessionNotificationCause cause, String attributeName, boolean local)
    {
-      return !ClusteredSessionNotificationCause.UNDEPLOY.equals(cause) 
+      return !ClusteredSessionNotificationCause.UNDEPLOY.equals(cause)
                && super.isHttpSessionAttributeListenerInvocationAllowed(status, cause, attributeName, local);
    }
 
    /**
     * Overrides superclass to return <code>false</code> if the cause of the
     * notification is {@link ClusteredSessionNotificationCause.UNDEPLOY}.
-    *  
+    *
     * @return <code>true</code> if <code>status.isLocallyUsed()</code>
     *         is <code>true</code> and the cause of the notification is not
     *         {@link ClusteredSessionNotificationCause.UNDEPLOY}.
@@ -57,8 +57,8 @@
    public boolean isHttpSessionListenerInvocationAllowed(ClusteredSessionManagementStatus status,
          ClusteredSessionNotificationCause cause, boolean local)
    {
-      return !ClusteredSessionNotificationCause.UNDEPLOY.equals(cause) 
-               && isHttpSessionListenerInvocationAllowed(status, cause, local);
+      return !ClusteredSessionNotificationCause.UNDEPLOY.equals(cause)
+               && super.isHttpSessionListenerInvocationAllowed(status, cause, local);
    }
 
 }