[wp-trac] [WordPress Trac] #22400: Remove all, or at least most, uses of extract() within WordPress

WordPress Trac noreply at wordpress.org
Thu May 15 20:05:46 UTC 2014


#22400: Remove all, or at least most, uses of extract() within WordPress
-------------------------------------------------+-------------------------
 Reporter:  Viper007Bond                         |       Owner:
     Type:  enhancement                          |      Status:  new
 Priority:  normal                               |   Milestone:  4.0
Component:  General                              |     Version:  3.4.2
 Severity:  normal                               |  Resolution:
 Keywords:  westi-likes needs-testing has-patch  |     Focuses:
  needs-unit-tests                               |
-------------------------------------------------+-------------------------
Changes (by wonderboymusic):

 * keywords:  westi-likes needs-testing has-patch => westi-likes needs-
     testing has-patch needs-unit-tests


Comment:

 [attachment:wp-delete-term.diff wp-delete-term.diff] requires unit tests.

 `wp_delete_term()` has a 3rd argument: `$args`.
 `$args` is an `array` that can look like `array( 'default' => 3,
 'force_default' => true )`

 None of our unit tests cover this scenario. I don't have the energy to
 write them right now. If someone else wants to: #patcheswelcome

--
Ticket URL: <https://core.trac.wordpress.org/ticket/22400#comment:99>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list