[wp-trac] [WordPress Trac] #3670: Removing CDATA close tag ( ]]> ) unbalances the CDATA block
WordPress Trac
wp-trac at lists.automattic.com
Sun Nov 14 03:20:32 UTC 2010
#3670: Removing CDATA close tag ( ]]> ) unbalances the CDATA block
--------------------------+-------------------------------------------------
Reporter: scenic | Owner: andy
Type: defect (bug) | Status: new
Priority: normal | Milestone: Future Release
Component: Template | Version: 2.1
Severity: minor | Keywords: has-patch
--------------------------+-------------------------------------------------
Changes (by andy):
* priority: high => normal
* severity: major => minor
Comment:
CDATA wrappers should be removed from all templates and applied via
filters using new functions. There are some complications.
A quick search for 'the_title_rss' (e.g.) finds that filter tag used in
wp-admin/includes/export.php, which reminds that we use feed-related
functions for more than just feeds. Any changes to feed-related filters
must be tested with export/import.
The same search also finds the template tag the_title_rss() in feed-
rss2.php and feed-atom.php, but only CDATA-wrapped in the latter.
Universal use of CDATA tags to guarantee XML validity is of dubious merit.
However, if we are going to do it we should be consistent.
3670-I.diff is the first half of the fix. It will break some feeds (rare)
so it should not be applied by itself. The second half, installing new
CDATA filters, touches a lot of code. Please volunteer.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/3670#comment:57>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list