You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Werner Punz (Issue Comment Edited) (JIRA)" <de...@myfaces.apache.org> on 2011/09/29 14:44:45 UTC
[jira] [Issue Comment Edited] (MYFACES-3339) jsf.js:render all
corner case fails on IE and chrome
[ https://issues.apache.org/jira/browse/MYFACES-3339?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13117200#comment-13117200 ]
Werner Punz edited comment on MYFACES-3339 at 9/29/11 12:43 PM:
----------------------------------------------------------------
Ok a client side fix does not make sense there. This needs to be resolved on partial response writer level.
The myfaces ppr should introduce CDATA escapes aka the script should become:
<script type="text/javascript">
// <CDATA[[
var a && b;
// ]]>
</script>
<update id="..."><CDATA[[...
<script="text/javascript">
// <!CDATA[[
// ]]]]><!CDATA[[>
</script>
]]></update>
This is perfectly viable since more than one cdata section can follow the update.
I remember that I once introduced such an escape code in our codebase, I wonder why it is gone.
I have to forward this to leo to check why we do not escape cdata sections anymore in our PPR.
was (Author: werpu):
Ok a client side fix does not make sense there. This needs to be resolved on partial response writer level.
The myfaces ppr should introduce CDATA escapes aka the script should become:
<script type="text/javascript">
<CDATA[[
var a && b;
]]>
</script>
<update id="..."><CDATA[[...
<script="text/javascript">
<!CDATA[[
]]]]><!CDATA[[>
</script>
]]></update>
This is perfectly viable since more than one cdata section can follow the update.
I remember that I once introduced such an escape code in our codebase, I wonder why it is gone.
I have to forward this to leo to check why we do not escape cdata sections anymore in our PPR.
> jsf.js:render all corner case fails on IE and chrome
> ----------------------------------------------------
>
> Key: MYFACES-3339
> URL: https://issues.apache.org/jira/browse/MYFACES-3339
> Project: MyFaces Core
> Issue Type: Bug
> Reporter: Werner Punz
>
> One of my automated tests regarding render javax.faces.ViewRoot fails on chrome and ie, it works on all other browsers.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira