Make WordPress Core


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

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

Improve cache key generation in the WP_Network_Query class by removing update_network_cache element in the array used to generate the cache key. This
element does not affect that cache and by removing it, it improves the likelihood of reusing an existing cache.

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

File:
1 edited

Legend:

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

    r51869 r53098  
    511511
    512512        /**
     513         * @ticket 55461
     514         */
     515        public function test_wp_network_query_cache_with_same_fields_same_cache_field() {
     516            $q                 = new WP_Network_Query();
     517            $query_1           = $q->query(
     518                array(
     519                    'fields'               => 'all',
     520                    'number'               => 3,
     521                    'order'                => 'ASC',
     522                    'update_network_cache' => true,
     523                )
     524            );
     525            $number_of_queries = get_num_queries();
     526
     527            $query_2 = $q->query(
     528                array(
     529                    'fields'               => 'all',
     530                    'number'               => 3,
     531                    'order'                => 'ASC',
     532                    'update_network_cache' => true,
     533                )
     534            );
     535
     536            $this->assertSame( $number_of_queries, get_num_queries() );
     537        }
     538
     539        /**
     540         * @ticket 55461
     541         */
     542        public function test_wp_network_query_cache_with_same_fields_different_cache_field() {
     543            $q                 = new WP_Network_Query();
     544            $query_1           = $q->query(
     545                array(
     546                    'fields'               => 'all',
     547                    'number'               => 3,
     548                    'order'                => 'ASC',
     549                    'update_network_cache' => true,
     550                )
     551            );
     552            $number_of_queries = get_num_queries();
     553
     554            $query_2 = $q->query(
     555                array(
     556                    'fields'               => 'all',
     557                    'number'               => 3,
     558                    'order'                => 'ASC',
     559                    'update_network_cache' => false,
     560                )
     561            );
     562
     563            $this->assertSame( $number_of_queries, get_num_queries() );
     564        }
     565
     566        /**
    513567         * @ticket 45749
    514568         * @ticket 47599
Note: See TracChangeset for help on using the changeset viewer.