[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