[wp-trac] [WordPress Trac] #32257: Patch: add support for multi-line textarea sanitization
WordPress Trac
noreply at wordpress.org
Mon Oct 24 01:32:13 UTC 2016
#32257: Patch: add support for multi-line textarea sanitization
--------------------------------------+------------------------
Reporter: ottok | Owner: chriscct7
Type: enhancement | Status: reviewing
Priority: normal | Milestone: 4.7
Component: Formatting | Version:
Severity: normal | Resolution:
Keywords: has-patch has-unit-tests | Focuses:
--------------------------------------+------------------------
Comment (by pento):
Thank you for your patience with this ticket, @ottok. Your persistence is
greatly appreciated. :-)
A minor workflow note - I appreciate that GitHub is your preferred
development environment, but for the purposes of reviewing changes,
patches ''must'' be uploaded to Trac - it's currently the only way we can
ensure that code hasn't changed between a review being requested, and the
review happening.
Anyway, on to the change! In [attachment:32257.2.diff]:
* Convert the unit tests to a `dataProvider` model, and adds a few more
tests.
* Minor documentation and code tweaks.
I'm concerned about the tag stripping behaviour, I've included a
deliberately failing unit test to demonstrate - a lone `<` isn't being
converted to `>`.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/32257#comment:17>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list