WordPress.org

Make WordPress Core

Ticket #37217: 37217.3.diff

File 37217.3.diff, 5.2 KB (added by flixos90, 3 years ago)
  • src/wp-includes/class-wp-network.php

     
    267267         * @return WP_Network|bool Network object if successful. False when no network is found.
    268268         */
    269269        public static function get_by_path( $domain = '', $path = '', $segments = null ) {
    270                 global $wpdb;
    271 
    272270                $domains = array( $domain );
    273271                $pieces  = explode( '.', $domain );
    274272
     
    295293                if ( wp_using_ext_object_cache() ) {
    296294                        $using_paths = wp_cache_get( 'networks_have_paths', 'site-options' );
    297295                        if ( false === $using_paths ) {
    298                                 $using_paths = (int) $wpdb->get_var( "SELECT id FROM {$wpdb->site} WHERE path <> '/' LIMIT 1" );
     296                                $using_paths = get_networks( array(
     297                                        'number'       => 1,
     298                                        'count'        => true,
     299                                        'path__not_in' => '/',
     300                                ) );
    299301                                wp_cache_add( 'networks_have_paths', $using_paths, 'site-options'  );
    300302                        }
    301303                }
     
    356358                // @todo Consider additional optimization routes, perhaps as an opt-in for plugins.
    357359                // We already have paths covered. What about how far domains should be drilled down (including www)?
    358360
    359                 $search_domains = "'" . implode( "', '", $wpdb->_escape( $domains ) ) . "'";
    360 
    361361                if ( ! $using_paths ) {
    362                         $network = $wpdb->get_row( "
    363                                 SELECT * FROM {$wpdb->site}
    364                                 WHERE domain IN ({$search_domains})
    365                                 ORDER BY CHAR_LENGTH(domain)
    366                                 DESC LIMIT 1
    367                         " );
     362                        $networks = get_networks( array(
     363                                'number'     => 1,
     364                                'orderby'    => array(
     365                                        'domain_length' => 'DESC',
     366                                ),
     367                                'domain__in' => $domains,
     368                        ) );
    368369
    369                         if ( ! empty( $network ) && ! is_wp_error( $network ) ) {
    370                                 return new WP_Network( $network );
     370                        if ( ! empty( $networks ) ) {
     371                                return array_shift( $networks );
    371372                        }
    372373
    373374                        return false;
    374 
    375                 } else {
    376                         $search_paths = "'" . implode( "', '", $wpdb->_escape( $paths ) ) . "'";
    377                         $networks = $wpdb->get_results( "
    378                                 SELECT * FROM {$wpdb->site}
    379                                 WHERE domain IN ({$search_domains})
    380                                 AND path IN ({$search_paths})
    381                                 ORDER BY CHAR_LENGTH(domain) DESC, CHAR_LENGTH(path) DESC
    382                         " );
    383375                }
    384376
     377                $networks = get_networks( array(
     378                        'orderby'    => array(
     379                                'domain_length' => 'DESC',
     380                                'path_length'   => 'DESC',
     381                        ),
     382                        'domain__in' => $domains,
     383                        'path__in'   => $paths,
     384                ) );
     385
    385386                /*
    386387                 * Domains are sorted by length of domain, then by length of path.
    387388                 * The domain must match for the path to be considered. Otherwise,
     
    402403                }
    403404
    404405                if ( true === $found ) {
    405                         return new WP_Network( $network );
     406                        return $network;
    406407                }
    407408
    408409                return false;
  • tests/phpunit/tests/multisite/bootstrap.php

     
    8989        }
    9090
    9191        /**
     92         * @ticket 37217
     93         * @dataProvider data_get_network_by_path_not_using_paths
     94         *
     95         * @param string $expected_key The array key associated with expected data for the test.
     96         * @param string $domain       The requested domain.
     97         * @param string $path         The requested path.
     98         * @param string $message      The message to pass for failed tests.
     99         */
     100        public function test_get_network_by_path_not_using_paths( $expected_key, $domain, $path, $message ) {
     101                // Temporarily store original object cache and using paths values.
     102                $using_ext_object_cache_orig = wp_using_ext_object_cache( true );
     103                if ( $using_ext_object_cache_orig ) {
     104                        $using_paths_orig = wp_cache_get( 'networks_have_paths', 'site-options' );
     105                }
     106
     107                wp_cache_set( 'networks_have_paths', 0, 'site-options'  );
     108
     109                $network = get_network_by_path( $domain, $path );
     110
     111                // Restore original object cache and using paths values.
     112                if ( ! $using_ext_object_cache_orig ) {
     113                        wp_using_ext_object_cache( $using_ext_object_cache_orig );
     114                } else {
     115                        wp_cache_set( 'networks_have_paths', $using_paths_orig, 'site-options'  );
     116                }
     117
     118                $this->assertEquals( self::$network_ids[ $expected_key ], $network->id, $message );
     119        }
     120
     121        public function data_get_network_by_path_not_using_paths() {
     122                return array(
     123                        array( 'wordpress.org/',         'wordpress.org',       '/',          'A standard domain and path request should work.' ),
     124                        array( 'wordpress.net/',         'wordpress.net',       '/notapath/', 'A network matching a top level domain should be found regardless of path.' ),
     125                        array( 'www.wordpress.net/',     'www.wordpress.net',   '/notapath/', 'A network matching a domain should be found regardless of path.' ),
     126                        array( 'wordpress.org/',         'www.wordpress.org',   '/one/',      'Should find the network despite the www and regardless of path.' ),
     127                        array( 'wordpress.org/',         'site1.wordpress.org', '/one/',      'Should find the network with the corresponding top level domain regardless of path.' ),
     128                        array( 'www.wordpress.net/',     'www.wordpress.net',   '/two/',      'A www network can coexist with a non-www network.' ),
     129                        array( 'make.wordpress.org/',    'make.wordpress.org',  '/notapath/', 'A subdomain network should be found regardless of path.' ),
     130                );
     131        }
     132
     133        /**
    92134         * @ticket 27003
    93135         * @ticket 27927
    94136         * @dataProvider data_get_site_by_path