[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