You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by hl...@apache.org on 2011/07/05 20:53:08 UTC
svn commit: r1143167 - in
/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services:
ClusteredSessionImpl.java SessionFactory.java SessionFactoryImpl.java
Author: hlship
Date: Tue Jul 5 18:53:07 2011
New Revision: 1143167
URL: http://svn.apache.org/viewvc?rev=1143167&view=rev
Log:
Update some comments on new code
Modified:
tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/ClusteredSessionImpl.java
tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionFactory.java
tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionFactoryImpl.java
Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/ClusteredSessionImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/ClusteredSessionImpl.java?rev=1143167&r1=1143166&r2=1143167&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/ClusteredSessionImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/ClusteredSessionImpl.java Tue Jul 5 18:53:07 2011
@@ -23,6 +23,8 @@ import java.util.Map;
/**
* A thin wrapper around {@link javax.servlet.http.HttpSession}.
+ *
+ * @since 5.3
*/
public class ClusteredSessionImpl extends SessionImpl
{
Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionFactory.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionFactory.java?rev=1143167&r1=1143166&r2=1143167&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionFactory.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionFactory.java Tue Jul 5 18:53:07 2011
@@ -16,6 +16,13 @@ package org.apache.tapestry5.internal.se
import org.apache.tapestry5.services.Session;
+/**
+ * Encapsulates logic for accessing the {@link Session} (a wrapper around the {@link javax.servlet.http.HttpSession}).
+ *
+ * @since 5.3
+ * @see SessionImpl
+ * @see ClusteredSessionImpl
+ */
public interface SessionFactory
{
Session getSession(boolean create);
Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionFactoryImpl.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionFactoryImpl.java?rev=1143167&r1=1143166&r2=1143167&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionFactoryImpl.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/SessionFactoryImpl.java Tue Jul 5 18:53:07 2011
@@ -22,10 +22,6 @@ import org.apache.tapestry5.services.Ses
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpSession;
-/**
- * User: josh_canfield
- * Date: 6/25/11
- */
public class SessionFactoryImpl implements SessionFactory
{
private boolean clustered;
@@ -47,7 +43,10 @@ public class SessionFactoryImpl implemen
{
final HttpSession httpSession = request.getSession(create);
- if (httpSession == null ) return null;
+ if (httpSession == null)
+ {
+ return null;
+ }
if (clustered)
{