[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