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

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

Author: jiwils
Date: 2008-12-18 13:58:14 -0500 (Thu, 18 Dec 2008)
New Revision: 82381

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

Modified: branches/Branch_5_0/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/IgnoreUndeployLegacyClusteredSessionNotificationPolicy.java
===================================================================
--- branches/Branch_5_0/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/IgnoreUndeployLegacyClusteredSessionNotificationPolicy.java	2008-12-18 18:57:43 UTC (rev 82380)
+++ branches/Branch_5_0/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/IgnoreUndeployLegacyClusteredSessionNotificationPolicy.java	2008-12-18 18:58:14 UTC (rev 82381)
@@ -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);
    }
 
 }