Edgewall Software

Opened 7 years ago

Last modified 5 years ago

#12677 closed defect

glitch in ITemplateStreamFilter backward compatibility support — at Initial Version

Reported by: Christian Boos Owned by: Christian Boos
Priority: normal Milestone: 1.3.4
Component: rendering Version: 1.3dev
Severity: normal Keywords: jinja2 genshi ITemplateFilter
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

On an instance running r15747 where the SpamFilter is active, the (render with Jinja2 / parse as HTML / apply filters / re-render with Genshi) render cycle is broken for CDATA sections, as can be seen on the /about page:

  <div id="systeminfo">
    <script type="text/javascript">//&lt;![CDATA[
      jQuery(document).ready(function ($) {
        $("#systeminfo table")
          .before("&lt;p&gt;User Agent: &lt;code&gt;" + navigator.userAgent + "&lt;/code&gt;&lt;/p&gt;")
          .append("&lt;tr&gt;&lt;th&gt;jQuery&lt;/th&gt;&lt;td&gt;" + $().jquery + "&lt;/td&gt;&lt;/tr&gt;" +
                  "&lt;tr&gt;&lt;th&gt;jQuery UI&lt;/th&gt;&lt;td&gt;" + $.ui.version + "&lt;/td&gt;&lt;/tr&gt;" +
                  "&lt;tr&gt;&lt;th&gt;jQuery Timepicker&lt;/th&gt;&lt;td&gt;" + $.timepicker.version +
                  "&lt;/td&gt;&lt;/tr&gt;");
      });
    //]]&gt;</script>

It should be:

  <div id="systeminfo">
    <script type="text/javascript">//<![CDATA[
      jQuery(document).ready(function ($) {
        $("#systeminfo table")
          .before("<p>User Agent: <code>" + navigator.userAgent + "</code></p>")
          .append("<tr><th>jQuery</th><td>" + $().jquery + "</td></tr>" +
                  "<tr><th>jQuery UI</th><td>" + $.ui.version + "</td></tr>" +
                  "<tr><th>jQuery Timepicker</th><td>" + $.timepicker.version +
                  "</td></tr>");
      });
    //]]></script>

(rendered with Trac trunk at r15474 as well, but with normal Jinja2 rendering)

Change History (0)

Note: See TracTickets for help on using tickets.