Changeset 52814 for trunk/tests/phpunit/tests/wp/parseRequest.php

Timestamp:

03/02/2022 03:10:48 PM (3 years ago)

Author:

spacedmonkey

Message:

Bootstrap/Load: Stop unnecessary queries when using the do_parse_request filter.

Developers of plugins and themes can use the do_parse_request filter to hot-wire requests and hook in early to render custom pages. However, even through these request may not need post queries and 404 lookups to be run, they run anyway. This can results in unnecessary SQL queries running on these requests. By adding a return value to the parse_request method of the WP class, these queries can now be skipped.

Props junsuijin, ryan, westi, sivel, dd32, wonderboymusic, arnee, tyxla, DrewAPicture, lukecavanagh, SergeyBiryukov, davidbaumwald, Spacedmonkey, pbearne.
Fixes #10886.

File:

• trunk/tests/phpunit/tests/wp/parseRequest.php (modified) (1 diff)

trunk/tests/phpunit/tests/wp/parseRequest.php

@@ -40 +40 @@
         $this->assertSame( '', $this->wp->request );
     }
+    /**
+     * Test that the parse_request() returns bool
+     *
+     * @ticket 10886
+     */
+    public function test_parse_request_returns_bool() {
+
+        // check if parse_request() returns true for default setup.
+        $this->assertTrue( $this->wp->parse_request(), 'returns true' );
+
+        // add filter to shortcut the parse_request function.
+        add_filter( 'do_parse_request', '__return_false' );
+        $this->assertFalse( $this->wp->parse_request(), 'returns false' );
+        remove_filter( 'do_parse_request', '__return_false' );
+
+    }
 }