You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Marcus Schmidke (JIRA)" <de...@myfaces.apache.org> on 2007/10/16 10:36:51 UTC

[jira] Created: (TOMAHAWK-1132) JavaScript variable myThemeOfficeBase isn't rendered in CDATA when using XHTML

JavaScript variable myThemeOfficeBase isn't rendered in CDATA when using XHTML
------------------------------------------------------------------------------

                 Key: TOMAHAWK-1132
                 URL: https://issues.apache.org/jira/browse/TOMAHAWK-1132
             Project: MyFaces Tomahawk
          Issue Type: Bug
          Components: JS Cook Menu
    Affects Versions: 1.1.7-SNAPSHOT
         Environment: JBoss 4.2
JBoss Seam 2.0
Facelets
tomahawk-facelets-1.1.6
            Reporter: Marcus Schmidke


The JavaScript variable "myThemeOfficeBase" which is used to declare the JSCookMenu theme isn't rendered in a CDATA section when using XHTML/facelets.

Generated source:
<script type="text/javascript"><!--
var myThemeOfficeBase='/Argos3Web/faces/myFacesExtensionResource/org.apache.myfaces.renderkit.html.util.MyFacesResourceLoader/11925214/navmenu.jscookmenu.HtmlJSCookMenuRenderer/ThemeOffice/';
//--></script>

should look:
<script type="text/javascript">
//<![CDATA[
var myThemeOfficeBase='/Argos3Web/faces/myFacesExtensionResource/org.apache.myfaces.renderkit.html.util.MyFacesResourceLoader/11925214/navmenu.jscookmenu.HtmlJSCookMenuRenderer/ThemeOffice/';
//]]>
</script>

This is because DefaultAddResource.java, line 769, passes the wrong content type to HtmlResponseWriterImpl. However, passing the right content type caused a NullPointerException in HtmlRendererUtils.isAllowedCdataSection. This was already corrected for 1.1.7, but I don't know whether it will work that way, because it now simply returns false.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.