[wp-trac] [WordPress Trac] #37392: Multisite "Sites" screen: Add links to filter websites by status
WordPress Trac
noreply at wordpress.org
Tue Sep 17 15:40:33 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-refresh has- | Focuses: multisite
patch |
-------------------------------------------------+-------------------------
Comment (by johnjamesjacoby):
37392.9.patch is the same as 37392.diff with the following changes:
* Cleaned up some indentations that had both spaces and tabs 😔
* Cleaned up copy-pasta that referred to "role" instead of "status"
* Cleaned up `wp_count_sites()` by reordering some invocations, and
limiting`WP_Site_Query()` object creation down to 1 object instead of one
for every status
I think this is great, and a future iteration should not have the statuses
hardcoded. See #12706.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/37392#comment:40>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list