WordPress.org

Make WordPress Core

Ticket #37181: 37181.3.diff

File 37181.3.diff, 14.2 KB (added by spacedmonkey, 3 years ago)
  • src/wp-includes/class-wp-network-query.php

     
    111111         *     @type array        $path__not_in         Array of paths to exclude affiliated networks for. Default empty.
    112112         *     @type string       $search               Search term(s) to retrieve matching networks for. Default empty.
    113113         *     @type bool         $update_network_cache Whether to prime the cache for found networks. Default true.
     114     *     @type bool         $update_network_meta_cache Whether to prime the metadata (option) cache for found networks.
     115     *                                                   Default true.
    114116         * }
    115117         */
    116118        public function __construct( $query = '' ) {
     
    132134                        'path__not_in'         => '',
    133135                        'search'               => '',
    134136                        'update_network_cache' => true,
     137                        'update_network_meta_cache' => true,
    135138                );
    136139
    137140                if ( ! empty( $query ) ) {
     
    242245                }
    243246
    244247                if ( $this->query_vars['update_network_cache'] ) {
    245                         _prime_network_caches( $network_ids );
     248                        _prime_network_caches( $network_ids, $this->query_vars['update_network_meta_cache'] );
    246249                }
    247250
    248251                // Fetch full network objects from the primed cache.
  • src/wp-includes/load.php

     
    542542
    543543                require( ABSPATH . WPINC . '/kses.php' );
    544544                require( ABSPATH . WPINC . '/pluggable.php' );
    545                 require( ABSPATH . WPINC . '/formatting.php' );
    546545
    547546                $link = wp_guess_url() . '/wp-admin/install.php';
    548547
  • src/wp-includes/ms-blogs.php

     
    11701170
    11711171        foreach ( (array) $ids as $id ) {
    11721172                wp_cache_delete( $id, 'networks' );
     1173                wp_cache_delete( $id, 'site_meta' );
    11731174
    11741175                /**
    11751176                 * Fires immediately after a network has been removed from the object cache.
     
    11921193 * cache using the network group with the key using the ID of the networks.
    11931194 *
    11941195 * @since 4.6.0
     1196 * @since 4.9.0 Introduced the `$update_meta_cache` parameter.
    11951197 *
    11961198 * @param array $networks Array of network row objects.
     1199 * @param bool  $update_meta_cache Whether to update sitemeta cache. Default true.
    11971200 */
    1198 function update_network_cache( $networks ) {
     1201function update_network_cache( $networks, $update_meta_cache = true ) {
     1202    $network_ids = array();
    11991203        foreach ( (array) $networks as $network ) {
     1204        $network_ids[] = $network->id;
    12001205                wp_cache_add( $network->id, $network, 'networks' );
    12011206        }
     1207
     1208    if ( $update_meta_cache ) {
     1209        update_meta_cache( 'site', $network_ids );
     1210    }
    12021211}
    12031212
    12041213/**
    12051214 * Adds any networks from the given IDs to the cache that do not already exist in cache.
    12061215 *
    12071216 * @since 4.6.0
     1217 * @since 4.9.0 Introduced the `$update_meta_cache` parameter.
    12081218 * @access private
    12091219 *
    12101220 * @see update_network_cache()
    12111221 * @global wpdb $wpdb WordPress database abstraction object.
    12121222 *
    12131223 * @param array $network_ids Array of network IDs.
     1224 * @param bool  $update_meta_cache Optional. Whether to update the meta cache. Default true.
    12141225 */
    1215 function _prime_network_caches( $network_ids ) {
     1226function _prime_network_caches( $network_ids, $update_meta_cache = true ) {
    12161227        global $wpdb;
    12171228
    12181229        $non_cached_ids = _get_non_cached_ids( $network_ids, 'networks' );
    12191230        if ( !empty( $non_cached_ids ) ) {
    12201231                $fresh_networks = $wpdb->get_results( sprintf( "SELECT $wpdb->site.* FROM $wpdb->site WHERE id IN (%s)", join( ",", array_map( 'intval', $non_cached_ids ) ) ) );
    12211232
    1222                 update_network_cache( $fresh_networks );
     1233                update_network_cache( $fresh_networks, $update_meta_cache );
    12231234        }
    12241235}
    12251236
  • src/wp-includes/option.php

     
    214214 *
    215215 * @global wpdb $wpdb WordPress database abstraction object.
    216216 *
    217  * @param int $site_id Optional site ID for which to query the options. Defaults to the current site.
     217 * @param int $network_id Optional site ID for which to query the options. Defaults to the current site.
    218218 */
    219 function wp_load_core_site_options( $site_id = null ) {
     219function wp_load_core_site_options( $network_id = null ) {
    220220        global $wpdb;
    221221
    222222        if ( ! is_multisite() || wp_using_ext_object_cache() || wp_installing() )
    223223                return;
    224224
    225         if ( empty($site_id) )
    226                 $site_id = $wpdb->siteid;
     225    $network_id = (int) $network_id;
    227226
    228         $core_options = array('site_name', 'siteurl', 'active_sitewide_plugins', '_site_transient_timeout_theme_roots', '_site_transient_theme_roots', 'site_admins', 'can_compress_scripts', 'global_terms_enabled', 'ms_files_rewriting' );
     227    // Fallback to the current network if a network ID is not specified.
     228    if ( ! $network_id ) {
     229        $network_id = get_current_network_id();
     230    }
    229231
    230         $core_options_in = "'" . implode("', '", $core_options) . "'";
    231         $options = $wpdb->get_results( $wpdb->prepare("SELECT meta_key, meta_value FROM $wpdb->sitemeta WHERE meta_key IN ($core_options_in) AND site_id = %d", $site_id) );
    232 
    233         foreach ( $options as $option ) {
    234                 $key = $option->meta_key;
    235                 $cache_key = "{$site_id}:$key";
    236                 $option->meta_value = maybe_unserialize( $option->meta_value );
    237 
    238                 wp_cache_set( $cache_key, $option->meta_value, 'site-options' );
    239         }
     232        update_meta_cache( 'site', $network_id );
    240233}
    241234
    242235/**
     
    11401133                return $pre;
    11411134        }
    11421135
    1143         // prevent non-existent options from triggering multiple queries
    1144         $notoptions_key = "$network_id:notoptions";
    1145         $notoptions = wp_cache_get( $notoptions_key, 'site-options' );
    1146 
    1147         if ( isset( $notoptions[ $option ] ) ) {
    1148 
    1149                 /**
    1150                  * Filters a specific default network option.
    1151                  *
    1152                  * The dynamic portion of the hook name, `$option`, refers to the option name.
    1153                  *
    1154                  * @since 3.4.0
    1155                  * @since 4.4.0 The `$option` parameter was added.
    1156                  * @since 4.7.0 The `$network_id` parameter was added.
    1157                  *
    1158                  * @param mixed  $default    The value to return if the site option does not exist
    1159                  *                           in the database.
    1160                  * @param string $option     Option name.
    1161                  * @param int    $network_id ID of the network.
    1162                  */
    1163                 return apply_filters( "default_site_option_{$option}", $default, $option, $network_id );
    1164         }
    11651136
    11661137        if ( ! is_multisite() ) {
    11671138                /** This filter is documented in wp-includes/option.php */
    11681139                $default = apply_filters( 'default_site_option_' . $option, $default, $option, $network_id );
    11691140                $value = get_option( $option, $default );
    11701141        } else {
    1171                 $cache_key = "$network_id:$option";
    1172                 $value = wp_cache_get( $cache_key, 'site-options' );
    1173 
    1174                 if ( ! isset( $value ) || false === $value ) {
    1175                         $row = $wpdb->get_row( $wpdb->prepare( "SELECT meta_value FROM $wpdb->sitemeta WHERE meta_key = %s AND site_id = %d", $option, $network_id ) );
    1176 
    1177                         // Has to be get_row instead of get_var because of funkiness with 0, false, null values
    1178                         if ( is_object( $row ) ) {
    1179                                 $value = $row->meta_value;
    1180                                 $value = maybe_unserialize( $value );
    1181                                 wp_cache_set( $cache_key, $value, 'site-options' );
    1182                         } else {
    1183                                 if ( ! is_array( $notoptions ) ) {
    1184                                         $notoptions = array();
    1185                                 }
    1186                                 $notoptions[ $option ] = true;
    1187                                 wp_cache_set( $notoptions_key, $notoptions, 'site-options' );
    1188 
    1189                                 /** This filter is documented in wp-includes/option.php */
    1190                                 $value = apply_filters( 'default_site_option_' . $option, $default, $option, $network_id );
    1191                         }
    1192                 }
     1142        $meta = get_metadata( 'site', $network_id, $option );
     1143        if ( is_array( $meta ) && ! empty( $meta ) ) {
     1144            $value = array_shift( $meta );
     1145            $value = maybe_unserialize( $value );
     1146        } else {
     1147            $value = apply_filters( 'default_site_option_' . $option, $default, $option );
     1148        }
    11931149        }
    11941150
    11951151        /**
     
    12571213         */
    12581214        $value = apply_filters( "pre_add_site_option_{$option}", $value, $option, $network_id );
    12591215
    1260         $notoptions_key = "$network_id:notoptions";
    1261 
    12621216        if ( ! is_multisite() ) {
    12631217                $result = add_option( $option, $value, '', 'no' );
    12641218        } else {
    1265                 $cache_key = "$network_id:$option";
    1266 
    1267                 // Make sure the option doesn't already exist. We can check the 'notoptions' cache before we ask for a db query
    1268                 $notoptions = wp_cache_get( $notoptions_key, 'site-options' );
    1269                 if ( ! is_array( $notoptions ) || ! isset( $notoptions[ $option ] ) ) {
    1270                         if ( false !== get_network_option( $network_id, $option, false ) ) {
    1271                                 return false;
    1272                         }
    1273                 }
    1274 
    1275                 $value = sanitize_option( $option, $value );
    1276 
    1277                 $serialized_value = maybe_serialize( $value );
    1278                 $result = $wpdb->insert( $wpdb->sitemeta, array( 'site_id'    => $network_id, 'meta_key'   => $option, 'meta_value' => $serialized_value ) );
    1279 
    1280                 if ( ! $result ) {
    1281                         return false;
    1282                 }
    1283 
    1284                 wp_cache_set( $cache_key, $value, 'site-options' );
    1285 
    1286                 // This option exists now
    1287                 $notoptions = wp_cache_get( $notoptions_key, 'site-options' ); // yes, again... we need it to be fresh
    1288                 if ( is_array( $notoptions ) && isset( $notoptions[ $option ] ) ) {
    1289                         unset( $notoptions[ $option ] );
    1290                         wp_cache_set( $notoptions_key, $notoptions, 'site-options' );
    1291                 }
     1219                $value  = sanitize_option( $option, $value );
     1220                $value  = maybe_serialize( $value );
     1221                $result = add_metadata( 'site', $network_id, $option, $value, true );
    12921222        }
    12931223
    12941224        if ( $result ) {
     
    13701300        if ( ! is_multisite() ) {
    13711301                $result = delete_option( $option );
    13721302        } else {
    1373                 $row = $wpdb->get_row( $wpdb->prepare( "SELECT meta_id FROM {$wpdb->sitemeta} WHERE meta_key = %s AND site_id = %d", $option, $network_id ) );
    1374                 if ( is_null( $row ) || ! $row->meta_id ) {
    1375                         return false;
    1376                 }
    1377                 $cache_key = "$network_id:$option";
    1378                 wp_cache_delete( $cache_key, 'site-options' );
    1379 
    1380                 $result = $wpdb->delete( $wpdb->sitemeta, array( 'meta_key' => $option, 'site_id' => $network_id ) );
     1303                $result = delete_metadata( 'site', $network_id, $option, '' );
    13811304        }
    13821305
    13831306        if ( $result ) {
     
    14701393                return add_network_option( $network_id, $option, $value );
    14711394        }
    14721395
    1473         $notoptions_key = "$network_id:notoptions";
    1474         $notoptions = wp_cache_get( $notoptions_key, 'site-options' );
    1475         if ( is_array( $notoptions ) && isset( $notoptions[ $option ] ) ) {
    1476                 unset( $notoptions[ $option ] );
    1477                 wp_cache_set( $notoptions_key, $notoptions, 'site-options' );
    1478         }
    1479 
    14801396        if ( ! is_multisite() ) {
    14811397                $result = update_option( $option, $value, 'no' );
    14821398        } else {
    1483                 $value = sanitize_option( $option, $value );
    1484 
    1485                 $serialized_value = maybe_serialize( $value );
    1486                 $result = $wpdb->update( $wpdb->sitemeta, array( 'meta_value' => $serialized_value ), array( 'site_id' => $network_id, 'meta_key' => $option ) );
    1487 
    1488                 if ( $result ) {
    1489                         $cache_key = "$network_id:$option";
    1490                         wp_cache_set( $cache_key, $value, 'site-options' );
    1491                 }
     1399                $value  = sanitize_option( $option, $value );
     1400                $value  = maybe_serialize( $value );
     1401                $result = update_metadata( 'site', $network_id, $option, $value );
    14921402        }
    14931403
    14941404        if ( $result ) {
  • src/wp-settings.php

     
    9595// Load early WordPress files.
    9696require( ABSPATH . WPINC . '/compat.php' );
    9797require( ABSPATH . WPINC . '/class-wp-list-util.php' );
     98require( ABSPATH . WPINC . '/formatting.php' );
     99require( ABSPATH . WPINC . '/meta.php' );
    98100require( ABSPATH . WPINC . '/functions.php' );
    99101require( ABSPATH . WPINC . '/class-wp-matchesmapregex.php' );
    100102require( ABSPATH . WPINC . '/class-wp.php' );
     
    142144// Load most of WordPress.
    143145require( ABSPATH . WPINC . '/class-wp-walker.php' );
    144146require( ABSPATH . WPINC . '/class-wp-ajax-response.php' );
    145 require( ABSPATH . WPINC . '/formatting.php' );
     147
    146148require( ABSPATH . WPINC . '/capabilities.php' );
    147149require( ABSPATH . WPINC . '/class-wp-roles.php' );
    148150require( ABSPATH . WPINC . '/class-wp-role.php' );
     
    157159require( ABSPATH . WPINC . '/class-wp-user-query.php' );
    158160require( ABSPATH . WPINC . '/class-wp-session-tokens.php' );
    159161require( ABSPATH . WPINC . '/class-wp-user-meta-session-tokens.php' );
    160 require( ABSPATH . WPINC . '/meta.php' );
    161162require( ABSPATH . WPINC . '/class-wp-meta-query.php' );
    162163require( ABSPATH . WPINC . '/class-wp-metadata-lazyloader.php' );
    163164require( ABSPATH . WPINC . '/general-template.php' );
  • tests/phpunit/tests/option/multisite.php

     
    144144                //$this->assertFalse( get_option( $key2 ) ); // check get_option()
    145145        }
    146146
    147         /**
    148          * @group multisite
    149          */
    150         function test_site_notoptions() {
    151                 global $wpdb;
    152                 $notoptions_key = "{$wpdb->siteid}:notoptions";
    153 
    154                 $_notoptions = wp_cache_get( 'notoptions', 'site-options' );
    155                 $this->assertEmpty( $_notoptions );
    156                 $_notoptions1 = wp_cache_get( $notoptions_key, 'site-options' );
    157                 $this->assertEmpty( $_notoptions1 );
    158 
    159                 get_site_option( 'burrito' );
    160 
    161                 $notoptions = wp_cache_get( 'notoptions', 'site-options' );
    162                 $this->assertEmpty( $notoptions );
    163                 $notoptions1 = wp_cache_get( $notoptions_key, 'site-options' );
    164                 $this->assertNotEmpty( $notoptions1 );
    165         }
    166 
    167147        function test_users_can_register_signup_filter() {
    168148
    169149                $registration = get_site_option('registration');
  • tests/phpunit/tests/option/networkOption.php

     
    7878                $this->assertFalse( isset( $options[ $key ] ) );
    7979        }
    8080
     81    /**
     82     * @ticket 37181
     83     * @group ms-required
     84     */
     85        public function test_meta_api_use_in_network_option(){
     86        $network_id = self::factory()->network->create();
     87        $option = 'test_option_name';
     88        add_metadata( 'site', $network_id, $option, 'monday', true );
     89        add_metadata( 'site', $network_id, $option, 'tuesday', true );
     90        add_metadata( 'site', $network_id, $option, 'wednesday', true );
     91        $this->assertEquals( 'monday', get_network_option( $network_id, $option, true ) );
     92    }
     93
    8194        /**
    8295         * @dataProvider data_network_id_parameter
    8396         *