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

Timestamp:

10/18/2022 03:06:48 PM (3 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