Author: bdelacretaz
Date: Mon Dec  9 12:50:49 2013
New Revision: 1549565

URL: http://svn.apache.org/r1549565
Log:
Note problem with nested requests

Modified:
    
sling/whiteboard/bdelacretaz/request-context/src/main/java/org/apache/sling/requestcontext/api/SlingRequestContext.java

Modified: 
sling/whiteboard/bdelacretaz/request-context/src/main/java/org/apache/sling/requestcontext/api/SlingRequestContext.java
URL: 
http://svn.apache.org/viewvc/sling/whiteboard/bdelacretaz/request-context/src/main/java/org/apache/sling/requestcontext/api/SlingRequestContext.java?rev=1549565&r1=1549564&r2=1549565&view=diff
==============================================================================
--- 
sling/whiteboard/bdelacretaz/request-context/src/main/java/org/apache/sling/requestcontext/api/SlingRequestContext.java
 (original)
+++ 
sling/whiteboard/bdelacretaz/request-context/src/main/java/org/apache/sling/requestcontext/api/SlingRequestContext.java
 Mon Dec  9 12:50:49 2013
@@ -46,6 +46,11 @@ public class SlingRequestContext extends
     
     public SlingRequestContext(SlingHttpServletRequest request) {
         this.request = request;
+
+        // TODO with this we'll have a single request context
+        // shared between the "main" requests and the ones that
+        // are caused by including other content in it. Using 
+        // a stack of request contexts should fix that.
         request.setAttribute(this.getClass().getName(), this);
         
         put(USER_MAP_NAME, new HashMap<String, Object>());
@@ -102,4 +107,4 @@ public class SlingRequestContext extends
     public boolean isReadyOnly() {
         return readOnly;
     }
-}
\ No newline at end of file
+}


Reply via email to