Changeset 37692 for trunk/tests/phpunit/tests/post/query.php

Timestamp:

06/14/2016 01:59:25 AM (8 years ago)

Author:

boonebgorges

Message:

Query: Allow plugins to supply post results instead of having WP_Query fetch them from the database.

Returning a non-null value from the new posts_pre_query filter will cause
WP_Query to skip its database query, so that posts data can be provided from
elsewhere. This is useful in cases where post data may be mirrored in a
separate location, such as an external search application.

Developers should note that the WP_Query properties generally used to
calculate pagination - specifically, found_posts and max_num_pages, which
are determined by default in set_found_posts() - must be provided explicitly
when using the posts_pre_query filter; since WP_Query will not be
contacting the database, it will have no access to SELECT FOUND_ROWS().
The WP_Query instance is passed to posts_pre_query by reference, so that
these properties can be set manually if needed.

Props jpdavoutian, tlovett1.
Fixes #36687.

File:

• trunk/tests/phpunit/tests/post/query.php (modified) (1 diff)

trunk/tests/phpunit/tests/post/query.php

@@ -389 +389 @@
         $this->assertEqualSets( $requested, $actual_posts );
     }
+
+    /**
+     * @ticket 36687
+     */
+    public function test_posts_pre_query_filter_should_bypass_database_query() {
+        global $wpdb;
+
+        add_filter( 'posts_pre_query', array( __CLASS__, 'filter_posts_pre_query' ) );
+
+        $num_queries = $wpdb->num_queries;
+        $q = new WP_Query( array(
+            'fields' => 'ids',
+            'no_found_rows' => true,
+        ) );
+
+        remove_filter( 'posts_pre_query', array( __CLASS__, 'filter_posts_pre_query' ) );
+
+        $this->assertSame( $num_queries, $wpdb->num_queries );
+        $this->assertSame( array( 12345 ), $q->posts );
+    }
+
+    public static function filter_posts_pre_query( $posts ) {
+        return array( 12345 );
+    }
 }