WordPress.org
Get WordPress

Make WordPress Core

Ticket #36675: 36675.3.diff

File 36675.3.diff, 6.8 KB (added by jeremyfelt, 9 years ago)

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

     
    8383                $s = isset( $_REQUEST['s'] ) ? wp_unslash( trim( $_REQUEST[ 's' ] ) ) : '';
    8484                $wild = '';
    8585                if ( false !== strpos($s, '*') ) {
    86                         $wild = '%';
     86                        $wild = '*';
    8787                        $s = trim($s, '*');
    8888                }
    8989
    … …  
    9898                                $_GET['order'] = $_REQUEST['order'] = 'DESC';
    9999                }
    100100
    101                 $query = "SELECT * FROM {$wpdb->blogs} WHERE site_id = '{$wpdb->siteid}' ";
     101                $args = array(
     102                        'number'     => intval( $per_page ),
     103                        'offset'     => intval( ( $pagenum - 1 ) * $per_page ),
     104                        'network_id' => $current_site->id,
     105                );
    102106
    103107                if ( empty($s) ) {
    104108                        // Nothing to do.
    … …  
    107111                                        preg_match( '/^[0-9]{1,3}\.[0-9]{1,3}\.?$/', $s ) ||
    108112                                        preg_match( '/^[0-9]{1,3}\.$/', $s ) ) {
    109113                        // IPv4 address
    110                         $sql = $wpdb->prepare( "SELECT blog_id FROM {$wpdb->registration_log} WHERE {$wpdb->registration_log}.IP LIKE %s", $wpdb->esc_like( $s ) . $wild );
     114                        $sql = $wpdb->prepare( "SELECT blog_id FROM {$wpdb->registration_log} WHERE {$wpdb->registration_log}.IP LIKE %s", $wpdb->esc_like( $s ) . ( ! empty( $wild ) ? '%' : '' ) );
    111115                        $reg_blog_ids = $wpdb->get_col( $sql );
    112116
    113                         if ( !$reg_blog_ids )
    114                                 $reg_blog_ids = array( 0 );
    115 
    116                         $query = "SELECT *
    117                                 FROM {$wpdb->blogs}
    118                                 WHERE site_id = '{$wpdb->siteid}'
    119                                 AND {$wpdb->blogs}.blog_id IN (" . implode( ', ', $reg_blog_ids ) . ")";
     117                        if ( $reg_blog_ids ) {
     118                                $args['site__in'] = $reg_blog_ids;
     119                        }
    120120                } else {
    121                         if ( is_numeric($s) && empty( $wild ) ) {
    122                                 $query .= $wpdb->prepare( " AND ( {$wpdb->blogs}.blog_id = %s )", $s );
    123                         } elseif ( is_subdomain_install() ) {
    124                                 $blog_s = str_replace( '.' . $current_site->domain, '', $s );
    125                                 $blog_s = $wpdb->esc_like( $blog_s ) . $wild . $wpdb->esc_like( '.' . $current_site->domain );
    126                                 $query .= $wpdb->prepare( " AND ( {$wpdb->blogs}.domain LIKE %s ) ", $blog_s );
     121                        if ( is_numeric( $s ) && empty( $wild ) ) {
     122                                $args['ID'] = $s;
    127123                        } else {
    128                                 if ( $s != trim('/', $current_site->path) ) {
    129                                         $blog_s = $wpdb->esc_like( $current_site->path . $s ) . $wild . $wpdb->esc_like( '/' );
    130                                 } else {
    131                                         $blog_s = $wpdb->esc_like( $s );
    132                                 }
    133                                 $query .= $wpdb->prepare( " AND  ( {$wpdb->blogs}.path LIKE %s )", $blog_s );
     124                                $args['search'] = $s;
    134125                        }
    135126                }
    136127
    137128                $order_by = isset( $_REQUEST['orderby'] ) ? $_REQUEST['orderby'] : '';
    138                 if ( $order_by === 'registered' ) {
    139                         $query .= ' ORDER BY registered ';
    140                 } elseif ( $order_by === 'lastupdated' ) {
    141                         $query .= ' ORDER BY last_updated ';
    142                 } elseif ( $order_by === 'blogname' ) {
    143                         if ( is_subdomain_install() ) {
    144                                 $query .= ' ORDER BY domain ';
    145                         } else {
    146                                 $query .= ' ORDER BY path ';
    147                         }
    148                 } elseif ( $order_by === 'blog_id' ) {
    149                         $query .= ' ORDER BY blog_id ';
    150                 } else {
    151                         $order_by = null;
     129                if ( 'registered' === $order_by ) {
     130                        // registered is named properly.
     131                } elseif ( 'lastupdated' === $order_by ) {
     132                        $order_by = 'last_updated';
     133                } elseif ( 'blogname' === $order_by ) {
     134                        $order_by = 'domain';
     135                } elseif ( 'blog_id' === $order_by ) {
     136                        $order_by = 'id';
     137                } elseif ( ! $order_by ) {
     138                        $order_by = false;
    152139                }
    153140
    154                 if ( isset( $order_by ) ) {
    155                         $order = ( isset( $_REQUEST['order'] ) && 'DESC' === strtoupper( $_REQUEST['order'] ) ) ? "DESC" : "ASC";
    156                         $query .= $order;
     141                $args['orderby'] = $order_by;
     142
     143                if ( $order_by ) {
     144                        $args['order'] = ( isset( $_REQUEST['order'] ) && 'DESC' === strtoupper( $_REQUEST['order'] ) ) ? "DESC" : "ASC";
    157145                }
    158146
    159                 // Don't do an unbounded count on large networks
    160                 if ( ! wp_is_large_network() )
    161                         $total = $wpdb->get_var( str_replace( 'SELECT *', 'SELECT COUNT( blog_id )', $query ) );
     147                if ( wp_is_large_network() ) {
     148                        $args['no_found_rows'] = true;
     149                } else {
     150                        $args['no_found_rows'] = false;
     151                }
    162152
    163                 $query .= " LIMIT " . intval( ( $pagenum - 1 ) * $per_page ) . ", " . intval( $per_page );
    164                 $this->items = $wpdb->get_results( $query, ARRAY_A );
     153                $_sites = get_sites( $args );
     154                if ( is_array( $_sites ) ) {
     155                        update_site_cache( $_sites );
    165156
    166                 if ( wp_is_large_network() )
    167                         $total = count($this->items);
     157                        $this->items = array_slice( $_sites, 0, $per_page );
     158                }
    168159
     160                $total_sites = get_sites( array_merge( $args, array(
     161                        'count' => true,
     162                        'offset' => 0,
     163                        'number' => 0,
     164                ) ) );
     165
    169166                $this->set_pagination_args( array(
    170                         'total_items' => $total,
     167                        'total_items' => $total_sites,
    171168                        'per_page' => $per_page,
    172169                ) );
    173170        }
    … …  
    445442         */
    446443        public function display_rows() {
    447444                foreach ( $this->items as $blog ) {
     445                        $blog = $blog->to_array();
    448446                        $class = '';
    449447                        reset( $this->status_list );
    450448

  • src/wp-includes/class-wp-site-query.php

     
    563563        protected function get_search_sql( $string, $columns ) {
    564564                global $wpdb;
    565565
    566                 $like = '%' . $wpdb->esc_like( $string ) . '%';
     566                if ( false !== strpos( $string, '*' ) ) {
     567                        $like = '%' . implode( '%', array_map( array( $wpdb, 'esc_like' ), explode( '*', $string ) ) ) . '%';
     568                } else {
     569                        $like = '%' . $wpdb->esc_like( $string ) . '%';
     570                }
    567571
    568572                $searches = array();
    569573                foreach ( $columns as $column ) {

  • tests/phpunit/tests/multisite/wpMSSitesListTable.php

     
    7373
    7474                $expected = array(
    7575                        self::$site_ids['wordpress.org/foo/'],
     76                        self::$site_ids['wordpress.org/foo/bar/'],
     77                        self::$site_ids['wordpress.org/afoo/'],
    7678                        self::$site_ids['make.wordpress.org/foo/'],
    7779                        self::$site_ids['www.w.org/foo/'],
     80                        self::$site_ids['www.w.org/foo/bar/'],
    7881                );
    7982
    8083                $this->assertEqualSets( $expected, $items );
    … …  
    131134
    132135                $expected = array(
    133136                        self::$site_ids['test.example.org/'],
     137                        self::$site_ids['test2.example.org/'],
     138                        self::$site_ids['test3.example.org/zig/'],
     139                        self::$site_ids['atest.example.org/'],
    134140                );
    135141
    136142                $this->assertEqualSets( $expected, $items );
    … …  
    154160                        self::$site_ids['test.example.org/'],
    155161                        self::$site_ids['test2.example.org/'],
    156162                        self::$site_ids['test3.example.org/zig/'],
     163                        self::$site_ids['atest.example.org/'],
    157164                );
    158165
    159166                $this->assertEqualSets( $expected, $items );
    … …  
    176183                $expected = array(
    177184                        self::$site_ids['wordpress.org/foo/'],
    178185                        self::$site_ids['wordpress.org/foo/bar/'],
     186                        self::$site_ids['wordpress.org/afoo/'],
    179187                        self::$site_ids['make.wordpress.org/foo/'],
    180188                        self::$site_ids['www.w.org/foo/'],
    181189                        self::$site_ids['www.w.org/foo/bar/'],