[wp-trac] [WordPress Trac] #37392: Multisite "Sites" screen: Add links to filter websites by status
WordPress Trac
noreply at wordpress.org
Wed Jun 5 18:45:10 UTC 2019
#37392: Multisite "Sites" screen: Add links to filter websites by status
-------------------------------------------------+-------------------------
Reporter: thomaswm | Owner: mnelson4
Type: enhancement | Status: assigned
Priority: normal | Milestone: 5.3
Component: Networks and Sites | Version: 3.0
Severity: normal | Resolution:
Keywords: needs-unit-tests needs-patch needs- | Focuses: multisite
refresh |
-------------------------------------------------+-------------------------
Comment (by pbiron):
a couple of other minor comments about both 37392.diff and
37392.8.patch...
== 37392.diff
* there are spaces (instead of tab) in the DocBlock on `get_views()`
== 37392.8.patch
* the query in `wp_count_sites()` has `FROM wp_blogs` hardcoded and needs
to use `$wpdb->blogs`
== Both
* the inline comment in `prepare_items()` should say something like "Take
into account the site status the user has selected." instead of "Take into
account the role the user has selected."
--
Ticket URL: <https://core.trac.wordpress.org/ticket/37392#comment:35>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list