[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