Changeset 54634 for trunk/tests/phpunit/tests/query/cacheResults.php

Timestamp:

10/18/2022 03:06:48 PM (4 years ago)

Author:

spacedmonkey

Message:

Query: Remove placeholder from query cache key.

Remove escape placeholder from query cache key, as placeholders are a based on a unique id on every request. This means that it is impossible for a cache to be reused, making queries that use escape placeholders such as LIKE searches, unable to be cached.

Props dhl, spacedmonkey, peterwilsoncc, desrosj, chaion07, davidbaumwald, mukesh27.
Fixes #56802.

File:

• trunk/tests/phpunit/tests/query/cacheResults.php (modified) (6 diffs)

trunk/tests/phpunit/tests/query/cacheResults.php

@@ -33 +33 @@
          */
         public static $author_id;
+
+        /**
+         * @var array
+         */
+        protected $cache_args;
+
+        /**
+         * @var string
+         */
+        protected $new_request;
 
         public static function wpSetUpBeforeClass( WP_UnitTest_Factory $factory ) {
@@ -58 +68 @@
         }
 
+        function set_up() {
+                parent::set_up();
+                $this->cache_args  = null;
+                $this->new_request = null;
+                add_filter( 'wp_query_cache_key', array( $this, 'filter_wp_query_cache_key' ), 15, 3 );
+        }
+
         /**
          * @dataProvider data_query_cache
@@ -63 +80 @@
          */
         public function test_query_cache( $args ) {
-                $query1 = new WP_Query();
-                $posts1 = $query1->query( $args );
+                global $wpdb;
+
+                $query1 = new WP_Query();
+                $posts1 = $query1->query( $args );
+
+                $placeholder = $wpdb->placeholder_escape();
+                $this->assertNotEmpty( $this->new_request, 'Check new request is not empty' );
+                $this->assertStringNotContainsString( $placeholder, $this->new_request, 'Check if request does not contain placeholder' );
+                $this->assertStringNotContainsString( $placeholder, wp_json_encode( $this->cache_args ), 'Check if cache arrays does not contain placeholder' );
 
                 $queries_before = get_num_queries();
@@ -192 +216 @@
                                 ),
                         ),
+                        'cache meta query search'                     => array(
+                                'args' => array(
+                                        'cache_results' => true,
+                                        'meta_query'    => array(
+                                                array(
+                                                        'key'     => 'color',
+                                                        'value'   => '00',
+                                                        'compare' => 'LIKE',
+                                                ),
+                                        ),
+                                ),
+                        ),
+                        'cache meta query not search'                 => array(
+                                'args' => array(
+                                        'cache_results' => true,
+                                        'meta_query'    => array(
+                                                array(
+                                                        'key'     => 'color',
+                                                        'value'   => 'ff',
+                                                        'compare' => 'NOT LIKE',
+                                                ),
+                                        ),
+                                ),
+                        ),
                         'cache comment_count'                         => array(
                                 'args' => array(
@@ -208 +256 @@
                                                 ),
                                         ),
+                                ),
+                        ),
+                        'cache search query'                          => array(
+                                'args' => array(
+                                        'cache_results' => true,
+                                        's'             => 'title',
+                                ),
+                        ),
+                        'cache search query multiple terms'           => array(
+                                'args' => array(
+                                        'cache_results' => true,
+                                        's'             => 'Post title',
                                 ),
                         ),
@@ -828 +888 @@
         }
 
+        public function filter_wp_query_cache_key( $cache_key, $cache_args, $new_request ) {
+                $this->cache_args  = $cache_args;
+                $this->new_request = $new_request;
+
+                return $cache_key;
+        }
+
         /**
          * @ticket 22176