[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