WordPress.org
Get WordPress

Make WordPress Core

Ticket #27003: 27003.6.diff

File 27003.6.diff, 12.2 KB (added by nacin, 11 years ago)

Adds wp_load_core_site_options() calls to missing places

  • src/wp-includes/ms-load.php

     
    134134}
    135135
    136136/**
     137 * Retrieve a network object by its domain and path.
     138 *
     139 * @since 3.9.0
     140 *
     141 * @param string $domain Domain to check.
     142 * @param string $path   Path to check.
     143 * @return object|bool Network object if successful. False when no network is found.
     144 */
     145function get_network_by_path( $domain, $path ) {
     146        global $wpdb;
     147
     148        $network_id = false;
     149
     150        $domains = array( $domain );
     151        $pieces = explode( '.', $domain );
     152        // It's possible one domain to search is 'com', but it might as well
     153        // be 'localhost' or some other locally mapped domain.
     154        while ( array_shift( $pieces ) ) {
     155                if ( $pieces ) {
     156                        $domains[] = implode( '.', $pieces );
     157                }
     158        }
     159
     160        if ( '/' !== $path ) {
     161                $paths = array( '/', $path );
     162        } else {
     163                $paths = array( '/' );
     164        }
     165
     166        $domains = "'" . implode( "', '", esc_sql( $domains ) ) . "'";
     167        $paths = "'" . implode( "', '", esc_sql( $paths ) ) . "'";
     168
     169        $networks = $wpdb->get_results( "SELECT id, domain, path FROM $wpdb->site
     170                WHERE domain IN ($domains) AND path IN ($paths)
     171                ORDER BY CHAR_LENGTH(domain) DESC, CHAR_LENGTH(path) DESC" );
     172
     173        /*
     174         * Domains are sorted by length of domain, then by length of path.
     175         * The domain must match for the path to be considered. Otherwise,
     176         * a network with the path of / will suffice.
     177         */
     178        $found = false;
     179        foreach ( $networks as $network ) {
     180                if ( $network->domain === $domain ) {
     181                        if ( $network->path === $path ) {
     182                                $found = true;
     183                                break;
     184                        }
     185                        if ( $network->path === '/' ) {
     186                                $found = true;
     187                                break;
     188                        }
     189                }
     190        }
     191
     192        if ( $found ) {
     193                $network = wp_get_network( $network );
     194
     195                return $network;
     196        }
     197
     198        return false;
     199}
     200
     201/**
     202 * Retrieve an object containing information about the requested network.
     203 *
     204 * @since 3.9.0
     205 *
     206 * @param int $network_id The network's DB row or ID.
     207 * @return mixed Object containing network information if found, false if not.
     208 */
     209function wp_get_network( $network ) {
     210        global $wpdb;
     211
     212        if ( ! is_object( $network ) ) {
     213                $network = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->site WHERE id = %d", $network ) );
     214                if ( ! $network ) {
     215                        return false;
     216                }
     217        }
     218
     219        return $network;
     220}
     221
     222/**
    137223 * Sets current_site object.
    138224 *
    139225 * @access private
    … …  
    141227 * @return object $current_site object
    142228 */
    143229function wpmu_current_site() {
    144         global $wpdb, $current_site, $domain, $path, $sites, $cookie_domain;
     230        global $current_site, $domain, $path;
    145231
    146232        if ( empty( $current_site ) )
    147233                $current_site = new stdClass;
    … …  
    154240                        $current_site->blog_id = BLOG_ID_CURRENT_SITE;
    155241                elseif ( defined( 'BLOGID_CURRENT_SITE' ) ) // deprecated.
    156242                        $current_site->blog_id = BLOGID_CURRENT_SITE;
    157                 if ( DOMAIN_CURRENT_SITE == $domain )
    158                         $current_site->cookie_domain = $cookie_domain;
    159                 elseif ( substr( $current_site->domain, 0, 4 ) == 'www.' )
    160                         $current_site->cookie_domain = substr( $current_site->domain, 4 );
    161                 else
    162                         $current_site->cookie_domain = $current_site->domain;
    163243
    164244                wp_load_core_site_options( $current_site->id );
    165 
     245                $current_site = get_current_site_name( $current_site );
    166246                return $current_site;
    167247        }
    168248
    169249        $current_site = wp_cache_get( 'current_site', 'site-options' );
    170         if ( $current_site )
     250        if ( $current_site ) {
     251                wp_load_core_site_options();
     252                $current_site = get_current_site_name( $current_site );
    171253                return $current_site;
     254        }
    172255
    173         $sites = $wpdb->get_results( "SELECT * FROM $wpdb->site" ); // usually only one site
    174         if ( 1 == count( $sites ) ) {
    175                 $current_site = $sites[0];
     256        // One network is most common.
     257        $networks = $wpdb->get_col( "SELECT id FROM $wpdb->site LIMIT 2" );
     258        if ( count( $networks ) <= 1 ) {
     259                $current_site = wp_get_network( $networks[0]->id );
     260
    176261                wp_load_core_site_options( $current_site->id );
    177                 $path = $current_site->path;
    178                 $current_site->blog_id = $wpdb->get_var( $wpdb->prepare( "SELECT blog_id FROM $wpdb->blogs WHERE domain = %s AND path = %s", $current_site->domain, $current_site->path ) );
    179262                $current_site = get_current_site_name( $current_site );
    180                 if ( substr( $current_site->domain, 0, 4 ) == 'www.' )
    181                         $current_site->cookie_domain = substr( $current_site->domain, 4 );
    182                 wp_cache_set( 'current_site', $current_site, 'site-options' );
    183                 return $current_site;
    184         }
    185         $path = substr( $_SERVER[ 'REQUEST_URI' ], 0, 1 + strpos( $_SERVER[ 'REQUEST_URI' ], '/', 1 ) );
    186263
    187         if ( $domain == $cookie_domain )
    188                 $current_site = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->site WHERE domain = %s AND path = %s", $domain, $path ) );
    189         else
    190                 $current_site = $wpdb->get_row( $wpdb->prepare( "SELECT * FROM $wpdb->site WHERE domain IN ( %s, %s ) AND path = %s ORDER BY CHAR_LENGTH( domain ) DESC LIMIT 1", $domain, $cookie_domain, $path ) );
     264                $current_site->blog_id = $wpdb->get_var( $wpdb->prepare( "SELECT blog_id
     265                        FROM $wpdb->blogs WHERE domain = %s AND path = %s",
     266                        $current_site->domain, $current_site->path ) );
    191267
    192         if ( ! $current_site ) {
    193                 if ( $domain == $cookie_domain )
    194                         $current_site = $wpdb->get_row( $wpdb->prepare("SELECT * FROM $wpdb->site WHERE domain = %s AND path='/'", $domain ) );
    195                 else
    196                         $current_site = $wpdb->get_row( $wpdb->prepare("SELECT * FROM $wpdb->site WHERE domain IN ( %s, %s ) AND path = '/' ORDER BY CHAR_LENGTH( domain ) DESC LIMIT 1", $domain, $cookie_domain ) );
     268                wp_cache_set( 'current_site', 'site-options' );
    197269        }
    198270
    199         if ( $current_site ) {
    200                 $path = $current_site->path;
    201                 $current_site->cookie_domain = $cookie_domain;
    202                 return $current_site;
    203         }
     271        // Multiple networks in play. Determine which using the requested domain and path.
    204272
    205         if ( is_subdomain_install() ) {
    206                 $sitedomain = substr( $domain, 1 + strpos( $domain, '.' ) );
    207                 $current_site = $wpdb->get_row( $wpdb->prepare("SELECT * FROM $wpdb->site WHERE domain = %s AND path = %s", $sitedomain, $path) );
    208                 if ( $current_site ) {
    209                         $current_site->cookie_domain = $current_site->domain;
    210                         return $current_site;
    211                 }
     273        // Find the first complete path after the domain.
     274        $path = substr( $_SERVER[ 'REQUEST_URI' ], 0, 1 + strpos( $_SERVER[ 'REQUEST_URI' ], '/', 1 ) );
    212275
    213                 $current_site = $wpdb->get_row( $wpdb->prepare("SELECT * FROM $wpdb->site WHERE domain = %s AND path='/'", $sitedomain) );
    214         }
     276        $current_site = get_network_by_path( $domain, $path );
    215277
    216         if ( $current_site || defined( 'WP_INSTALLING' ) ) {
    217                 $path = '/';
     278        if ( $current_site ) {
     279                $path = $current_site->path;
     280                wp_load_core_site_options( $current_site->id );
     281                $current_site = get_current_site_name( $current_site );
    218282                return $current_site;
    219283        }
    220284
    221         // Still no dice.
    222285        wp_load_translations_early();
    223 
    224         if ( 1 == count( $sites ) )
    225                 wp_die( sprintf( __( 'That site does not exist. Please try <a href="%s">%s</a>.' ), 'http://' . $sites[0]->domain . $sites[0]->path ) );
    226         else
    227                 wp_die( __( 'No site defined on this host. If you are the owner of this site, please check <a href="http://codex.wordpress.org/Debugging_a_WordPress_Network">Debugging a WordPress Network</a> for help.' ) );
     286        wp_die( __( 'No site defined on this host. If you are the owner of this site, please check <a href="http://codex.wordpress.org/Debugging_a_WordPress_Network">Debugging a WordPress Network</a> for help.' ) );
    228287}
    229288
    230289/**

  • src/wp-includes/ms-settings.php

     
    3737        }
    3838
    3939        $domain = rtrim( $domain, '.' );
    40         $cookie_domain = $domain;
    41         if ( substr( $cookie_domain, 0, 4 ) == 'www.' )
    42                 $cookie_domain = substr( $cookie_domain, 4 );
    4340
    4441        $path = preg_replace( '|([a-z0-9-]+.php.*)|', '', $_SERVER['REQUEST_URI'] );
    4542        $path = str_replace ( '/wp-admin/', '/', $path );
    … …  
    4643        $path = preg_replace( '|(/[a-z0-9-]+?/).*|', '$1', $path );
    4744
    4845        $current_site = wpmu_current_site();
     46        $current_site->cookie_domain = $current_site->domain;
     47        if ( 'www.' === substr( $current_site->cookie_domain, 0, 4 ) ) {
     48                $current_site->cookie_domain = substr( $current_site->cookie_domain, 4 );
     49        }
     50
    4951        if ( ! isset( $current_site->blog_id ) )
    5052                $current_site->blog_id = $wpdb->get_var( $wpdb->prepare( "SELECT blog_id FROM $wpdb->blogs WHERE domain = %s AND path = %s", $current_site->domain, $current_site->path ) );
    5153

  • tests/phpunit/includes/factory.php

     
    4242         */
    4343        public $blog;
    4444
     45        /**
     46         * @var WP_UnitTest_Factory_For_Network
     47         */
     48        public $network;
     49
    4550        function __construct() {
    4651                $this->post = new WP_UnitTest_Factory_For_Post( $this );
    4752                $this->attachment = new WP_UnitTest_Factory_For_Attachment( $this );
    … …  
    5055                $this->term = new WP_UnitTest_Factory_For_Term( $this );
    5156                $this->category = new WP_UnitTest_Factory_For_Term( $this, 'category' );
    5257                $this->tag = new WP_UnitTest_Factory_For_Term( $this, 'post_tag' );
    53                 if ( is_multisite() )
     58                if ( is_multisite() ) {
    5459                        $this->blog = new WP_UnitTest_Factory_For_Blog( $this );
     60                        $this->network = new WP_UnitTest_Factory_For_Network( $this );
     61                }
    5562        }
    5663}
    5764
    … …  
    177184}
    178185
    179186
     187class WP_UnitTest_Factory_For_Network extends WP_UnitTest_Factory_For_Thing {
     188
     189        function __construct( $factory = null ) {
     190                parent::__construct( $factory );
     191                $this->default_generation_definitions = array(
     192                        'domain' => WP_TESTS_DOMAIN,
     193                        'title' => new WP_UnitTest_Generator_Sequence( 'Network %s' ),
     194                        'path' => new WP_UnitTest_Generator_Sequence( '/testpath%s/' ),
     195                        'network_id' => new WP_UnitTest_Generator_Sequence( '%s', 2 ),
     196                        'subdomain_install' => false,
     197                );
     198        }
     199
     200        function create_object( $args ) {
     201                require_once ABSPATH . 'wp-admin/includes/upgrade.php';
     202
     203                if ( ! isset( $args['user'] ) ) {
     204                        $email = WP_TESTS_EMAIL;
     205                } else {
     206                        $email = get_userdata( $args['user'] )->user_email;
     207                }
     208
     209                populate_network( $args['network_id'], $args['domain'], $email, $args['title'], $args['path'], $args['subdomain_install'] );
     210                return $args['network_id'];
     211        }
     212
     213        function update_object( $network_id, $fields ) {}
     214
     215        function get_object_by_id( $network_id ) {
     216                return wp_get_network( $network_id );
     217        }
     218}
     219
    180220class WP_UnitTest_Factory_For_Term extends WP_UnitTest_Factory_For_Thing {
    181221
    182222        private $taxonomy;

  • tests/phpunit/tests/ms.php

     
    11721172                // Expect 0 sites when using an offset larger than the number of sites
    11731173                $this->assertCount( 0, wp_get_sites( array( 'offset' => 20 ) ) );
    11741174        }
     1175
     1176        /**
     1177         * @ticket 27003
     1178         */
     1179        function test_get_network_by_path() {
     1180                global $wpdb;
     1181
     1182                $ids = array(
     1183                        'wordpress.org/'           => array( 'domain' => 'wordpress.org', 'path' => '/' ),
     1184                        'wordpress.net/'           => array( 'domain' => 'wordpress.net', 'path' => '/' ),
     1185                        'www.wordpress.net/'       => array( 'domain' => 'www.wordpress.net', 'path' => '/' ),
     1186                        'wordpress.org/one/'       => array( 'domain' => 'wordpress.org', 'path' => '/one/' ),
     1187                        'www.wordpress.org/two/'   => array( 'domain' => 'www.wordpress.org', 'path' => '/two/' ),
     1188                        'www.wordpress.net/three/' => array( 'domain' => 'www.wordpress.net', 'path' => '/three/' ),
     1189                        'wordpress.net/four/'      => array( 'domain' => 'wordpress.net', 'path' => '/four/' ),
     1190                );
     1191
     1192                foreach ( $ids as &$id ) {
     1193                        $id = $this->factory->network->create( $id );
     1194                }
     1195                unset( $id );
     1196
     1197                $this->assertEquals( $ids['www.wordpress.net/'],
     1198                        get_network_by_path( 'www.wordpress.net', '/notapath/' )->id );
     1199
     1200                $this->assertEquals( $ids['www.wordpress.net/three/'],
     1201                        get_network_by_path( 'www.wordpress.net', '/three/' )->id );
     1202
     1203                // This won't find /four/ because of the www.
     1204                $this->assertEquals( $ids['www.wordpress.net/'],
     1205                        get_network_by_path( 'www.wordpress.net', '/four/' )->id );
     1206
     1207                $this->assertEquals( $ids['wordpress.net/four/'],
     1208                        get_network_by_path( 'wordpress.net', '/four/' )->id );
     1209
     1210                $this->assertEquals( $ids['wordpress.net/'],
     1211                        get_network_by_path( 'wordpress.net', '/notapath/' )->id );
     1212        }
    11751213}
    11761214
    11771215endif;