Changeset 11591

Timestamp:

02/22/2022 03:46:33 AM (3 years ago)

Author:

dd32

Message:

Openverse iframe theme: Do not parse the request URI to generate a WP_Query, do not run FOUND_ROWS().

Parsing the request or trying to select the number of affected rows isn't needed here, as it's all passed to the iframe.

File:

• sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-openverse/functions.php (modified) (2 diffs)

sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-openverse/functions.php

@@ -54 +54 @@
  * disable those queries by returning an empty SQL
  * query for the main WP_Query instance.
+ * 
+ * Set no_found_rows to avoid querying for FOUND_ROWS().
  */
 function disable_default_query( $sql, $query ) {
     if ( $query->is_main_query() ) {
         $sql = '';
+        $query->set( 'no_found_rows', true );
     }
 
@@ -63 +66 @@
 }
 add_filter( 'posts_request', __NAMESPACE__ . '\disable_default_query', 10, 2 );
+
+/**
+ * Disable request parsing.
+ * 
+ * This avoids `WP` parsing the requested URI to generate the WP_Query parameters.
+ * As all URIs will be passed through to the iframe, there's no point in parsing
+ * this or querying for posts that definitely do not exist within the WordPress site.
+ */
+add_filter( 'do_parse_request', '__return_false' );
 
 /**