Changeset 37736 for trunk/src/wp-admin/includes/class-wp-ms-sites-list-table.php

Timestamp:

06/17/2016 12:02:05 AM (10 years ago)

Author:

jeremyfelt

Message:

Multisite: Use WP_Site_Query to power WP_MS_Sites_List_Table.

WP_Site_Query provides for a cleaner prepare_items() method. It significantly improves the search experience in the sites list table:

• In a subdomain configuration, domain and path are searched for a provided terms.
• In a subdirectory configuration, path is searched for a provided term.
• The full domain is searched in a subdomain configuration rather than the portion not matching the network's domain.
• Terms are searched as %term% by default. Adding * in the middle of a term will search %te%rm%.

Props flixos90, Fab1en.
Fixes #33185, #24833, #21837, #36675.

File:

• trunk/src/wp-admin/includes/class-wp-ms-sites-list-table.php (modified) (5 diffs)

trunk/src/wp-admin/includes/class-wp-ms-sites-list-table.php

@@ -68 +68 @@
                 global $s, $mode, $wpdb;
 
-                $current_site = get_current_site();
-
                 if ( ! empty( $_REQUEST['mode'] ) ) {
                         $mode = $_REQUEST['mode'] === 'excerpt' ? 'excerpt' : 'list';
@@ -84 +82 @@
                 $wild = '';
                 if ( false !== strpos($s, '*') ) {
-                        $wild = '%';
+                        $wild = '*';
                         $s = trim($s, '*');
                 }
@@ -99 +97 @@
                 }
 
-                $query = "SELECT * FROM {$wpdb->blogs} WHERE site_id = '{$wpdb->siteid}' ";
+                $args = array(
+                        'number'     => intval( $per_page ),
+                        'offset'     => intval( ( $pagenum - 1 ) * $per_page ),
+                        'network_id' => get_current_network_id(),
+                );
 
                 if ( empty($s) ) {
@@ -108 +110 @@
                                         preg_match( '/^[0-9]{1,3}\.$/', $s ) ) {
                         // IPv4 address
-                        $sql = $wpdb->prepare( "SELECT blog_id FROM {$wpdb->registration_log} WHERE {$wpdb->registration_log}.IP LIKE %s", $wpdb->esc_like( $s ) . $wild );
+                        $sql = $wpdb->prepare( "SELECT blog_id FROM {$wpdb->registration_log} WHERE {$wpdb->registration_log}.IP LIKE %s", $wpdb->esc_like( $s ) . ( ! empty( $wild ) ? '%' : '' ) );
                         $reg_blog_ids = $wpdb->get_col( $sql );
 
-                        if ( !$reg_blog_ids )
-                                $reg_blog_ids = array( 0 );
-
-                        $query = "SELECT *
-                                FROM {$wpdb->blogs}
-                                WHERE site_id = '{$wpdb->siteid}'
-                                AND {$wpdb->blogs}.blog_id IN (" . implode( ', ', $reg_blog_ids ) . ")";
+                        if ( $reg_blog_ids ) {
+                                $args['site__in'] = $reg_blog_ids;
+                        }
+                } elseif ( is_numeric( $s ) && empty( $wild ) ) {
+                        $args['ID'] = $s;
                 } else {
-                        if ( is_numeric($s) && empty( $wild ) ) {
-                                $query .= $wpdb->prepare( " AND ( {$wpdb->blogs}.blog_id = %s )", $s );
-                        } elseif ( is_subdomain_install() ) {
-                                $blog_s = str_replace( '.' . $current_site->domain, '', $s );
-                                $blog_s = $wpdb->esc_like( $blog_s ) . $wild . $wpdb->esc_like( '.' . $current_site->domain );
-                                $query .= $wpdb->prepare( " AND ( {$wpdb->blogs}.domain LIKE %s ) ", $blog_s );
+                        $args['search'] = $s;
+
+                        if ( ! is_subdomain_install() ) {
+                                $args['search_columns'] = array( 'path' );
+                        }
+                }
+
+                $order_by = isset( $_REQUEST['orderby'] ) ? $_REQUEST['orderby'] : '';
+                if ( 'registered' === $order_by ) {
+                        // registered is a valid field name.
+                } elseif ( 'lastupdated' === $order_by ) {
+                        $order_by = 'last_updated';
+                } elseif ( 'blogname' === $order_by ) {
+                        if ( is_subdomain_install() ) {
+                                $order_by = 'domain';
                         } else {
-                                if ( $s != trim('/', $current_site->path) ) {
-                                        $blog_s = $wpdb->esc_like( $current_site->path . $s ) . $wild . $wpdb->esc_like( '/' );
-                                } else {
-                                        $blog_s = $wpdb->esc_like( $s );
-                                }
-                                $query .= $wpdb->prepare( " AND  ( {$wpdb->blogs}.path LIKE %s )", $blog_s );
-                        }
-                }
-
-                $order_by = isset( $_REQUEST['orderby'] ) ? $_REQUEST['orderby'] : '';
-                if ( $order_by === 'registered' ) {
-                        $query .= ' ORDER BY registered ';
-                } elseif ( $order_by === 'lastupdated' ) {
-                        $query .= ' ORDER BY last_updated ';
-                } elseif ( $order_by === 'blogname' ) {
-                        if ( is_subdomain_install() ) {
-                                $query .= ' ORDER BY domain ';
-                        } else {
-                                $query .= ' ORDER BY path ';
-                        }
-                } elseif ( $order_by === 'blog_id' ) {
-                        $query .= ' ORDER BY blog_id ';
+                                $order_by = 'path';
+                        }
+                } elseif ( 'blog_id' === $order_by ) {
+                        $order_by = 'id';
+                } elseif ( ! $order_by ) {
+                        $order_by = false;
+                }
+
+                $args['orderby'] = $order_by;
+
+                if ( $order_by ) {
+                        $args['order'] = ( isset( $_REQUEST['order'] ) && 'DESC' === strtoupper( $_REQUEST['order'] ) ) ? "DESC" : "ASC";
+                }
+
+                if ( wp_is_large_network() ) {
+                        $args['no_found_rows'] = true;
                 } else {
-                        $order_by = null;
-                }
-
-                if ( isset( $order_by ) ) {
-                        $order = ( isset( $_REQUEST['order'] ) && 'DESC' === strtoupper( $_REQUEST['order'] ) ) ? "DESC" : "ASC";
-                        $query .= $order;
-                }
-
-                // Don't do an unbounded count on large networks
-                if ( ! wp_is_large_network() )
-                        $total = $wpdb->get_var( str_replace( 'SELECT *', 'SELECT COUNT( blog_id )', $query ) );
-
-                $query .= " LIMIT " . intval( ( $pagenum - 1 ) * $per_page ) . ", " . intval( $per_page );
-                $this->items = $wpdb->get_results( $query, ARRAY_A );
-
-                if ( wp_is_large_network() )
-                        $total = count($this->items);
+                        $args['no_found_rows'] = false;
+                }
+
+                $_sites = get_sites( $args );
+                if ( is_array( $_sites ) ) {
+                        update_site_cache( $_sites );
+
+                        $this->items = array_slice( $_sites, 0, $per_page );
+                }
+
+                $total_sites = get_sites( array_merge( $args, array(
+                        'count' => true,
+                        'offset' => 0,
+                        'number' => 0,
+                ) ) );
 
                 $this->set_pagination_args( array(
-                        'total_items' => $total,
+                        'total_items' => $total_sites,
                         'per_page' => $per_page,
                 ) );
@@ -446 +447 @@
         public function display_rows() {
                 foreach ( $this->items as $blog ) {
+                        $blog = $blog->to_array();
                         $class = '';
                         reset( $this->status_list );