Skip to content

Commit

Permalink
https://github.com/eclipse-ee4j/mojarra/issues/5164
Browse files Browse the repository at this point in the history
remove check whether the response buffer size is set so that the default
value can properly be used
  • Loading branch information
BalusC committed Nov 12, 2022
1 parent 6d40d92 commit 118c6c9
Showing 1 changed file with 2 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,6 @@ public class FaceletViewHandlingStrategy extends ViewHandlingStrategy {
private MethodRetargetHandlerManager retargetHandlerManager = new MethodRetargetHandlerManager();

private int responseBufferSize;
private boolean responseBufferSizeSet;

private Cache<Resource, BeanInfo> metadataCache;
private Map<String, List<String>> contractMappings;
Expand Down Expand Up @@ -812,7 +811,6 @@ protected void initialize() {
});

try {
responseBufferSizeSet = webConfig.isSet(FaceletsBufferSize);
responseBufferSize = Integer.parseInt(webConfig.getOptionValue(FaceletsBufferSize));
} catch (NumberFormatException nfe) {
responseBufferSize = Integer.parseInt(FaceletsBufferSize.getDefaultValue());
Expand Down Expand Up @@ -890,10 +888,8 @@ protected ResponseWriter createResponseWriter(FacesContext context) throws IOExc
throw new IllegalStateException("No render kit was available for id \"" + id + "\"");
}

if (responseBufferSizeSet) {
// set the buffer for content
extContext.setResponseBufferSize(responseBufferSize);
}
// set the buffer for content
extContext.setResponseBufferSize(responseBufferSize);

// get our content type
String contentType = (String) context.getAttributes().get("facelets.ContentType");
Expand Down

0 comments on commit 118c6c9

Please sign in to comment.