Make WordPress Core


Ignore:
Timestamp:
04/07/2022 05:44:34 PM (2 years ago)
Author:
spacedmonkey
Message:

Networks and Sites: Improve cache key generation in WP_Site_Query class.

Improve cache key generation in the WP_Site_Query class by removing update_site_cache and update_site_meta_cache elements in the array used to generate
the cache key. These elements do not affect that cache and by removing them, improve the likelihood of reusing an existing cache.

Props Spacedmonkey, furi3r, johnbillion, johnjamesjacoby, flixos90.
Fixes #55462.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/tests/phpunit/tests/multisite/wpSiteQuery.php

    r51869 r53097  
    850850            );
    851851
    852             $number_of_queries = $wpdb->num_queries;
     852            $number_of_queries = get_num_queries();
    853853
    854854            $query_2 = $q->query(
     
    862862            );
    863863            $this->assertSame( $number_of_queries + 1, $wpdb->num_queries );
     864        }
     865
     866        /**
     867         * @ticket 55462
     868         */
     869        public function test_wp_site_query_cache_with_same_fields_same_cache_fields() {
     870            $q = new WP_Site_Query();
     871
     872            $query_1 = $q->query(
     873                array(
     874                    'fields'                 => 'ids',
     875                    'network_id'             => self::$network_ids['wordpress.org/'],
     876                    'number'                 => 3,
     877                    'order'                  => 'ASC',
     878                    'update_site_cache'      => true,
     879                    'update_site_meta_cache' => true,
     880                )
     881            );
     882
     883            $number_of_queries = get_num_queries();
     884
     885            $query_2 = $q->query(
     886                array(
     887                    'fields'                 => 'ids',
     888                    'network_id'             => self::$network_ids['wordpress.org/'],
     889                    'number'                 => 3,
     890                    'order'                  => 'ASC',
     891                    'update_site_cache'      => true,
     892                    'update_site_meta_cache' => true,
     893                )
     894            );
     895            $this->assertSame( $number_of_queries, get_num_queries() );
     896        }
     897
     898        /**
     899         * @ticket 55462
     900         */
     901        public function test_wp_site_query_cache_with_same_fields_different_cache_fields() {
     902            $q = new WP_Site_Query();
     903
     904            $query_1 = $q->query(
     905                array(
     906                    'fields'                 => 'ids',
     907                    'network_id'             => self::$network_ids['wordpress.org/'],
     908                    'number'                 => 3,
     909                    'order'                  => 'ASC',
     910                    'update_site_cache'      => true,
     911                    'update_site_meta_cache' => true,
     912                )
     913            );
     914
     915            $number_of_queries = get_num_queries();
     916
     917            $query_2 = $q->query(
     918                array(
     919                    'fields'                 => 'ids',
     920                    'network_id'             => self::$network_ids['wordpress.org/'],
     921                    'number'                 => 3,
     922                    'order'                  => 'ASC',
     923                    'update_site_cache'      => false,
     924                    'update_site_meta_cache' => false,
     925                )
     926            );
     927            $this->assertSame( $number_of_queries, get_num_queries() );
    864928        }
    865929
Note: See TracChangeset for help on using the changeset viewer.