Skip to main content

Source code revision

svn

summary: http://java.net/jira/browse/JAVASERVERFACES-3150
revision: 12793
author: edburns
date: 2014-01-24 16:10:37 UTC (3 years)
message: http://java.net/jira/browse/JAVASERVERFACES-3150

I assert the cause of this bug is the dontEscape ivar of
HtmlResponseWriter is too agressively set as a side effect in
isScriptOrStyle(String name). I don't even understand why the ivar is
set as a side effect, but I'm not going to change that. What I will do
is add an extra assignment of false to the value of that ivar in
endElement(String).

SECTION: Modified Files
----------------------------
M jsf-ri/src/main/java/com/sun/faces/renderkit/html_basic/HtmlResponseWriter.java

- in endElement(String) set dontEscape to false after writing the
closing script block.

M test/agnostic/facelets/processing/pom.xml
A test/agnostic/facelets/processing/escaping01
A test/agnostic/facelets/processing/escaping01/src
A test/agnostic/facelets/processing/escaping01/src/test
A test/agnostic/facelets/processing/escaping01/src/test/java
A test/agnostic/facelets/processing/escaping01/src/test/java/com
A test/agnostic/facelets/processing/escaping01/src/test/java/com/sun
A test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces
A test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test
A test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test/agnostic
A test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test/agnostic/facelets
A test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test/agnostic/facelets/processing
A test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test/agnostic/facelets/processing/escaping01
A test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test/agnostic/facelets/processing/escaping01/Issue3150IT.java
A test/agnostic/facelets/processing/escaping01/src/main
A test/agnostic/facelets/processing/escaping01/src/main/java
A test/agnostic/facelets/processing/escaping01/src/main/java/com
A test/agnostic/facelets/processing/escaping01/src/main/java/com/sun
A test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces
A test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces/test
A test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces/test/agnostic
A test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces/test/agnostic/facelets
A test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces/test/agnostic/facelets/processing
A test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces/test/agnostic/facelets/processing/escaping01
A test/agnostic/facelets/processing/escaping01/src/main/resources
A test/agnostic/facelets/processing/escaping01/src/main/webapp
A test/agnostic/facelets/processing/escaping01/src/main/webapp/index.xhtml
A test/agnostic/facelets/processing/escaping01/src/main/webapp/WEB-INF
A test/agnostic/facelets/processing/escaping01/src/main/webapp/WEB-INF/web.xml
A test/agnostic/facelets/processing/escaping01/src/main/webapp/WEB-INF/glassfish-web.xml
A test/agnostic/facelets/processing/escaping01/src/main/webapp/WEB-INF/faces-config.xml
A test/agnostic/facelets/processing/escaping01/pom.xml
A test/agnostic/facelets/processing/escaping01/nbactions.xml

- New test content

Change Path Actions
M branches/MOJARRA_2_2X_ROLLING/jsf-ri/src/main/java/com/sun/faces/renderkit/html_basic/HtmlResponseWriter.java
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/nbactions.xml
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/pom.xml
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/java
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/java/com
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/java/com/sun
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces/test
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces/test/agnostic
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces/test/agnostic/facelets
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces/test/agnostic/facelets/processing
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/java/com/sun/faces/test/agnostic/facelets/processing/escaping01
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/resources
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/webapp
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/webapp/WEB-INF
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/webapp/WEB-INF/faces-config.xml
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/webapp/WEB-INF/glassfish-web.xml
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/webapp/WEB-INF/web.xml
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/main/webapp/index.xhtml
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test/java
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test/java/com
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test/java/com/sun
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test/agnostic
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test/agnostic/facelets
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test/agnostic/facelets/processing
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test/agnostic/facelets/processing/escaping01
A branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/escaping01/src/test/java/com/sun/faces/test/agnostic/facelets/processing/escaping01/Issue3150IT.java
M branches/MOJARRA_2_2X_ROLLING/test/agnostic/facelets/processing/pom.xml
 
 
Close
loading
Please Confirm
Close