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

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

Author: bstansberry at jboss.com
Date: 2008-09-20 17:50:34 -0400 (Sat, 20 Sep 2008)
New Revision: 78730

Modified:
   branches/Branch_4_2/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/ClusteredSessionManagementStatus.java
Log:
[JBAS-5778] Fix assertion; javadoc

Modified: branches/Branch_4_2/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/ClusteredSessionManagementStatus.java
===================================================================
--- branches/Branch_4_2/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/ClusteredSessionManagementStatus.java	2008-09-20 21:49:56 UTC (rev 78729)
+++ branches/Branch_4_2/tomcat/src/main/org/jboss/web/tomcat/service/session/notification/ClusteredSessionManagementStatus.java	2008-09-20 21:50:34 UTC (rev 78730)
@@ -35,12 +35,25 @@
    private final Boolean locallyActive;
    private final Boolean locallyOwned;
    
+   /**
+    * Create a new ClusteredSessionManagementStatus.
+    * 
+    * @param realId the id of the session, excluding any jvmRoute.
+    * @param locallyUsed whether the session has been provided to the 
+    *                    application on this node.
+    * @param locallyActive whether this node is the most recent one to 
+    *                      handle a request for the session; <code>null</code>
+    *                      if unknown
+    * @param locallyOwned  whether this node is the "owner" of the session,
+    *                      <code>null</code> if unknown or the concept of
+    *                      ownership is unsupported.
+    */
    public ClusteredSessionManagementStatus(String realId,
                                            boolean locallyUsed, 
                                            Boolean locallyActive, 
                                            Boolean locallyOwned)
    {
-      if (this.realId == null)
+      if (realId == null)
          throw new IllegalArgumentException("realId is null");
       
       this.realId = realId;
@@ -50,6 +63,17 @@
       // If we are locallyActive, we are locally owned
       this.locallyOwned = (Boolean.TRUE.equals(locallyActive) ? Boolean.TRUE : locallyOwned);
    }
+   
+   /**
+    * Gets the id of the session, excluding any jvmRoute that may have
+    * been appended if JK is used.
+    * 
+    * @return the id. Will not return <code>null</code>. 
+    */
+   public String getRealId()
+   {
+      return realId;
+   }
 
    /**
     * Gets whether an HttpSession object for the given session has been