[wp-trac] [WordPress Trac] #37392: Multisite "Sites" screen: Add links to filter websites by status
WordPress Trac
noreply at wordpress.org
Tue Jun 4 23:46:49 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 spacedmonkey):
I have taken a bash at refreshing the patch. See
[https://core.trac.wordpress.org/attachment/ticket/37392/37392.diff
37392.diff].
`wp_count_sites` did have caching but not cache invalidation. So I have
converted it to use `WP_Site_Query`. In the original patch, it used the
weird nested count queries found in,
[https://developer.wordpress.org/reference/functions/count_users/
count_users]. That query in count_users is a nightmare and doesn't scale.
See #38741 for more information. After working on that ticket, it easier
to do 6 small queries that one big query to get the data we need.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/37392#comment:33>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list