[wp-trac] [WordPress Trac] #21767: Remove stripslashes from API functions
WordPress Trac
wp-trac at lists.automattic.com
Thu Sep 20 17:53:39 UTC 2012
#21767: Remove stripslashes from API functions
-------------------------------------------------+-------------------------
Reporter: alexkingorg | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: Awaiting
Component: General | Review
Severity: normal | Version: trunk
Keywords: has-patch needs-testing needs-unit- | Resolution:
tests |
-------------------------------------------------+-------------------------
Comment (by alexkingorg):
Here's a first unit test that handles the post object. It exercises the
controller function (`edit_post()`) as well as the model functions
(`wp_insert_post()`, `wp_update_post()`).
https://gist.github.com/3756939
These tests fail without the patch being applied and succeed with the
patch applied, so they are working as expected.
Some feedback on the approach and coverage here would be really helpful.
Did I miss anything obvious? Are there any "should have" things that were
omitted?
Assuming this is the right approach, I'll work on making sure we get the
same sort of coverage on the other objects as well. We can track progress
here (still needs to be detailed for other objects at this time):
https://docs.google.com/spreadsheet/ccc?key=0AqJlMmZg82o_dGoxT2pvbTY2VmY3VWpscEZFb1dLSGc
--
Ticket URL: <http://core.trac.wordpress.org/ticket/21767#comment:18>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list