Apply suggestions from code review

Co-authored-by: Peter Wilson <[email protected]>

src/wp-includes/class-wp-query.php

@@ -3109,7 +3109,17 @@ public function get_posts() {
  $found_rows = 'SQL_CALC_FOUND_ROWS';
  }
 
- // Beginning of the string is on a new line to prevent leading whitespace. See https://core.trac.wordpress.org/ticket/56841.
+ /*
+ * Beginning of the string is on a new line to prevent leading whitespace. 
+ *
+ * The additional indentation of subsequent lines is to ensure the SQL
+ * queries are identical to those generated when splitting queries. This
+ * improves caching of the query by ensuring the same cache key is 
+ * generated for the same database queries functionally.
+ *
+ * See https://core.trac.wordpress.org/ticket/56841.
+ * See https://github.com/WordPress/wordpress-develop/pull/6393#issuecomment-2088217429
+ */
  $old_request =
  "SELECT $found_rows $distinct $fields
  FROM {$wpdb->posts} $join

tests/phpunit/tests/query/cacheResults.php

@@ -197,7 +197,7 @@ public function test_generate_cache_key_unregister_post_type() {
  unregister_post_type( 'wptests_pt' );
  $cache_key_2 = $reflection->invoke( $query1, $query_vars, $request1 );
 
- $this->assertNotSame( $cache_key_1, $cache_key_2, 'Cache key should differs when after unregister post type.' );
+ $this->assertNotSame( $cache_key_1, $cache_key_2, 'Cache key should differ after unregistering post type.' );
  }
 
  /**