[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