You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ol...@apache.org on 2015/01/16 08:44:23 UTC

svn commit: r1652354 - /sling/trunk/contrib/scripting/thymeleaf/src/main/java/org/apache/sling/scripting/thymeleaf/internal/processor/attr/SlingIncludeAttrProcessor.java

Author: olli
Date: Fri Jan 16 07:44:23 2015
New Revision: 1652354

URL: http://svn.apache.org/r1652354
Log:
SLING-4297 SLING-4298 align usage

Modified:
    sling/trunk/contrib/scripting/thymeleaf/src/main/java/org/apache/sling/scripting/thymeleaf/internal/processor/attr/SlingIncludeAttrProcessor.java

Modified: sling/trunk/contrib/scripting/thymeleaf/src/main/java/org/apache/sling/scripting/thymeleaf/internal/processor/attr/SlingIncludeAttrProcessor.java
URL: http://svn.apache.org/viewvc/sling/trunk/contrib/scripting/thymeleaf/src/main/java/org/apache/sling/scripting/thymeleaf/internal/processor/attr/SlingIncludeAttrProcessor.java?rev=1652354&r1=1652353&r2=1652354&view=diff
==============================================================================
--- sling/trunk/contrib/scripting/thymeleaf/src/main/java/org/apache/sling/scripting/thymeleaf/internal/processor/attr/SlingIncludeAttrProcessor.java (original)
+++ sling/trunk/contrib/scripting/thymeleaf/src/main/java/org/apache/sling/scripting/thymeleaf/internal/processor/attr/SlingIncludeAttrProcessor.java Fri Jan 16 07:44:23 2015
@@ -23,14 +23,12 @@ import java.io.IOException;
 import javax.servlet.RequestDispatcher;
 import javax.servlet.ServletException;
 
-import org.apache.commons.io.IOUtils;
 import org.apache.sling.api.SlingHttpServletRequest;
 import org.apache.sling.api.SlingHttpServletResponse;
 import org.apache.sling.api.request.RequestDispatcherOptions;
 import org.apache.sling.api.resource.Resource;
 import org.apache.sling.api.resource.ResourceUtil;
 import org.apache.sling.api.resource.SyntheticResource;
-import org.apache.sling.scripting.core.servlet.BufferedServletOutputStream;
 import org.apache.sling.scripting.core.servlet.CaptureResponseWrapper;
 import org.apache.sling.scripting.thymeleaf.internal.SlingWebContext;
 import org.apache.sling.scripting.thymeleaf.internal.dom.NodeUtil;
@@ -137,19 +135,14 @@ public class SlingIncludeAttrProcessor e
             }
 
             if (dispatcher != null) {
-                final String encoding = slingHttpServletResponse.getCharacterEncoding();
-                final BufferedServletOutputStream bsos = new BufferedServletOutputStream(encoding);
                 try {
-                    final CaptureResponseWrapper wrapper = new CaptureResponseWrapper(slingHttpServletResponse, bsos);
+                    final CaptureResponseWrapper wrapper = new CaptureResponseWrapper(slingHttpServletResponse);
                     dispatcher.include(slingHttpServletRequest, wrapper);
                     if (!wrapper.isBinaryResponse()) {
-                        wrapper.flushBuffer();
-                        return bsos.getBuffer();
+                        return wrapper.getCapturedCharacterResponse();
                     }
                 } catch (ServletException e) {
                     logger.error(e.getMessage(), e);
-                } finally {
-                    IOUtils.closeQuietly(bsos);
                 }
             } else {
                 logger.error("no request dispatcher: unable to include {}/'{}'", resource, path);