[
https://issues.apache.org/jira/browse/MYFACES-2836?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12891551#action_12891551
]
Bruno Aranda commented on MYFACES-2836:
---------------------------------------
Uhm, you are right... I probably ran the test without updating shared before
the second fix (adding the constants).
And that is the problem, the CDATA constants already present contained a line
return as part of the value, making some assertions fail in myfaces-impl. So
nothing is really broken...
I am going to add new constants and commit again...
> HtmlResponseWriterImpl does not check if a CDATA section is already opened
> when writing the content of a script
> ---------------------------------------------------------------------------------------------------------------
>
> Key: MYFACES-2836
> URL: https://issues.apache.org/jira/browse/MYFACES-2836
> Project: MyFaces Core
> Issue Type: Bug
> Components: JSR-252
> Affects Versions: 2.0.1
> Reporter: Bruno Aranda
> Assignee: Bruno Aranda
> Fix For: 2.0.2-SNAPSHOT
>
> Attachments: MUFACES-2836.patch
>
>
> Hi,
> When the default response writer writes a script, it includes CDATA tags
> around the body. If the script is being written inside another CDATA section
> for some reason (Primefaces does this in its partial response), then invalid
> XML is created.
> I attach patch for the fix to be reviewed, before I commit something not
> quite right again and we can discuss. Basically, a simple boolean is used to
> check if a CDATA is already open or not.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
