[wp-trac] [WordPress Trac] #21990: SimplePie: DOMDocument can be disabled

WordPress Trac noreply at wordpress.org
Thu Nov 22 00:09:47 UTC 2012


#21990: SimplePie: DOMDocument can be disabled
-------------------------------------------------+-------------------------
 Reporter:  nacin                                |       Owner:
     Type:  defect (bug)                         |  markjaquith
 Priority:  normal                               |      Status:  assigned
Component:  External Libraries                   |   Milestone:  3.5
 Severity:  blocker                              |     Version:  trunk
 Keywords:  has-patch needs-docs dev-feedback    |  Resolution:
  needs-testing needs-unit-tests                 |
-------------------------------------------------+-------------------------
Changes (by markjaquith):

 * keywords:  needs-patch => has-patch needs-docs dev-feedback needs-testing
     needs-unit-tests


Comment:

 First attempt at sublassing SimplePie_Sanitize. New file needed because
 the SP files are loaded conditionally, and I wanted to leave those
 pristine. Avoided the fancy registration stuff, because we're manually
 instantiating the sanitization object ourselves. So I just changed that.
 Only did very preliminary testing: both branches of logic seem to not
 break the dashboard RSS feed.

 Will need PHPDoc, security testing, and unit testing — just wanted to get
 a first effort up here ASAP.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/21990#comment:13>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list