[wp-trac] [WordPress Trac] #7723: Allow filtering by author on Manage > Posts

WordPress Trac noreply at wordpress.org
Wed Jul 10 07:27:04 UTC 2013


#7723: Allow filtering by author on Manage > Posts
-------------------------------------------------+-------------------------
 Reporter:  jeremyclarke                         |       Owner:  anonymous
     Type:  enhancement                          |      Status:  reopened
 Priority:  normal                               |   Milestone:  Future
Component:  Administration                       |  Release
 Severity:  normal                               |     Version:
 Keywords:  has-patch dev-feedback needs-        |  Resolution:
  testing                                        |
-------------------------------------------------+-------------------------

Comment (by DrewAPicture):

 [attachment:7723-01.diff] looks like a pretty good start.

 Just a couple of suggestions:

 * There's no need to break the ternary logic onto multiple lines such as
 in 234-237 in class-wp-posts-list-table.php. Core style is generally one
 line.
 * We should also be checking against `wp_is_large_network()` in the count-
 check to avoid performance issues for sites with > 10,000 users.
 * For the input placeholder and option value, we can use `esc_attr_e()`
 and `esc_html_e()` respectively for outputting the strings

--
Ticket URL: <http://core.trac.wordpress.org/ticket/7723#comment:18>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list