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

WordPress Trac noreply at wordpress.org
Tue May 13 05:36:23 UTC 2014


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

Comment (by wonderboymusic):

 In [changeset:"28390"]:
 {{{
 #!CommitTicketReference repository="" revision="28390"
 Eliminate use of `extract()` in
 `WP_Terms_List_Table::display_rows_or_placeholder()`:

 * Set variables for `$page` and `$number`
 * `list(...) = $this->get_column_info()` can be removed, as none of the
 variables returned are used.
 * `orderby` and `search` can be checked from `$args`, leaving no reason to
 extract

 See #22400.
 }}}

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


More information about the wp-trac mailing list