[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