[Yanel-commits] rev 54869 - public/yanel/trunk/src/impl/java/org/wyona/yanel/impl/resources/usecase/thread

[michi at wyona.com]

Author: michi
Date: 2010-11-22 21:08:00 +0100 (Mon, 22 Nov 2010)
New Revision: 54869

Modified:
   public/yanel/trunk/src/impl/java/org/wyona/yanel/impl/resources/usecase/thread/ThreadUsecaseResource.java
Log:
note about http session and thread ID added

Modified: public/yanel/trunk/src/impl/java/org/wyona/yanel/impl/resources/usecase/thread/ThreadUsecaseResource.java
===================================================================
--- public/yanel/trunk/src/impl/java/org/wyona/yanel/impl/resources/usecase/thread/ThreadUsecaseResource.java	2010-11-22 19:57:02 UTC (rev 54868)
+++ public/yanel/trunk/src/impl/java/org/wyona/yanel/impl/resources/usecase/thread/ThreadUsecaseResource.java	2010-11-22 20:08:00 UTC (rev 54869)
@@ -123,6 +123,7 @@
      * @return thread or null if the thread is not attached to the session. 
      */
     public UsecaseThread getThread() {
+        // TODO: Is it really a good idea to set/get the thread ID within the session, because what if the session expires?!
         HttpSession session = getEnvironment().getRequest().getSession();
         return UsecaseThread.getThreadFromSession(session, getThreadID());
     }