Perf compares.

Author: peterwilsoncc

tests/phpunit/tests/query/thePost.php

@@ -45,6 +45,78 @@ public function the_post() {
 	}
 }
 
+class WP_Query_Trunk extends WP_Query {
+	/**
+	 * Sets up the current post.
+	 *
+	 * Retrieves the next post, sets up the post, sets the 'in the loop'
+	 * property to true.
+	 *
+	 * @since 1.5.0
+	 *
+	 * @global WP_Post $post Global post object.
+	 */
+	public function the_post() {
+		global $post;
+
+		if ( ! $this->in_the_loop ) {
+			// Get post IDs to prime incomplete post objects.
+			$post_ids = array_reduce(
+				$this->posts,
+				function ( $carry, $post ) {
+					if ( is_numeric( $post ) && $post > 0 ) {
+						// Query for post ID.
+						$carry[] = $post;
+					}
+
+					if ( is_object( $post ) && isset( $post->ID ) ) {
+						// Query for object, either WP_Post or stdClass.
+						$carry[] = $post->ID;
+					}
+
+					return $carry;
+				},
+				array()
+			);
+			if ( $post_ids ) {
+				_prime_post_caches( $post_ids, $this->query_vars['update_post_term_cache'], $this->query_vars['update_post_meta_cache'] );
+			}
+			$post_objects = array_map( 'get_post', $post_ids );
+			update_post_author_caches( $post_objects );
+		}
+
+		$this->in_the_loop = true;
+		$this->before_loop = false;
+
+		if ( -1 === $this->current_post ) { // Loop has just started.
+			/**
+			 * Fires once the loop is started.
+			 *
+			 * @since 2.0.0
+			 *
+			 * @param WP_Query $query The WP_Query instance (passed by reference).
+			 */
+			do_action_ref_array( 'loop_start', array( &$this ) );
+		}
+
+		$post = $this->next_post();
+
+		// Ensure a full post object is available.
+		if ( $post instanceof stdClass ) {
+			// stdClass indicates that a partial post object was queried.
+			$post = get_post( $post->ID );
+		} elseif ( is_numeric( $post ) ) {
+			// Numeric indicates that only post IDs were queried.
+			$post = get_post( $post );
+		}
+
+		// Set up the global post object for the loop.
+		$this->setup_postdata( $post );
+	}
+}
+
+
+
 
 /**
  * @group query
@@ -384,91 +456,65 @@ public function test_post_preview_links_autosaves() {
 	 * @dataProvider data_the_loop_fields
 	 */
 	public function test_the_post_changes_perf( $fields ) {
-
-		// Prime options, etc in query
-		$query = new WP_Query(
-			array(
-				'fields'    => $fields,
-				'post_type' => 'page',
-				'posts_per_page' => -1,
-			)
-		);
-
-
-
-		wp_cache_flush_group( 'posts' );
-		wp_cache_flush_group( 'post-queries' );
-
-		$time_start        = microtime( true );
-		$num_queries_start = get_num_queries();
-
-
-
-		$query = new WP_Query(
-			array(
-				'fields'    => $fields,
-				'post_type' => 'page',
-				'posts_per_page' => -1,
-			)
-		);
-
-
-
-		$iterations = 1000;
-		while ( $iterations-- ) {
-			// Traverse the loop.
-			while ( $query->have_posts() ) {
-				$query->the_post();
-				get_post();
-				get_the_author();
+		$query_classes = array( 'WP_Query_67', 'WP_Query_Trunk', 'WP_Query' );
+		$do_iterations = 1000;
+		$results       = array();
+		foreach( $query_classes as $query_class ) {
+
+			$iterations = $do_iterations;
+			$start_time = microtime( true );
+			$start_queries = get_num_queries();
+
+			while ( $iterations-- ) {
+				// Flush the relevant caches.
+				wp_cache_flush_group( 'posts' );
+				wp_cache_flush_group( 'post_meta' );
+				wp_cache_flush_group( 'post-queries' );
+				wp_cache_flush_group( 'users' );
+				wp_cache_flush_group( 'user_meta' );
+
+				$query = new $query_class(
+					array(
+						'fields'    => $fields,
+						'post_type' => 'page',
+						'posts_per_page' => -1,
+					)
+				);
+
+				// Traverse the loop.
+				while ( $query->have_posts() ) {
+					$query->the_post();
+					get_post();
+					get_the_author();
+				}
 			}
 
-			$query->rewind_posts();
-			$query->in_the_loop = false;
-			wp_cache_flush_group( 'posts' );
-		}
-
-		$new_loop_time = microtime( true ) - $time_start;
-		$new_loop_queries = get_num_queries() - $num_queries_start;
-
-		wp_cache_flush_group( 'posts' );
-		wp_cache_flush_group( 'post-queries' );
-
-		$time_start		= microtime( true );
-		$num_queries_start = get_num_queries();
-
-		$iterations = 1000;
-		$query = new WP_Query_67(
-			array(
-				'fields'    => $fields,
-				'post_type' => 'page',
-				'posts_per_page' => -1,
-			)
-		);
-
-		while ( $iterations-- ) {
-			// Traverse the loop.
-			while ( $query->have_posts() ) {
-				$query->the_post();
-				get_post();
-				get_the_author();
-			}
+			$loop_time = microtime( true ) - $start_time;
+			$loop_queries = get_num_queries() - $start_queries;
+			$loop_queries_per_iteration = $loop_queries / $do_iterations;
 
-			$query->rewind_posts();
-			$query->in_the_loop = false;
-			wp_cache_flush_group( 'posts' );
+			// store results.
+			$results[ $fields ][ $query_class ] = array(
+				'loop_time' => $loop_time,
+				'loop_queries' => $loop_queries,
+				'loop_queries_per_iteration' => $loop_queries_per_iteration,
+			);
 		}
 
-
-		$old_loop_time = microtime( true ) - $time_start;
-		$old_loop_queries = get_num_queries() - $num_queries_start;
+		foreach( $results as &$result ) {
+			$result['Time Trunk - 67'] = $result['WP_Query_Trunk']['loop_time'] - $result['WP_Query_67']['loop_time'];
+			$result['Time Patch - 67'] = $result['WP_Query']['loop_time'] - $result['WP_Query_67']['loop_time'];
+			$result['Time Patch - Trunk'] = $result['WP_Query']['loop_time'] - $result['WP_Query_Trunk']['loop_time'];
+		}
 
 		// Dump the results.
 		ob_start();
-		var_dump( $fields );
-		var_dump( 'new', $new_loop_time, $new_loop_queries );
-		var_dump( 'old', $old_loop_time, $old_loop_queries );
+		var_dump( $results );
 		$output = ob_get_clean();
 		fwrite (STDERR, $output);
+
+
+		$this->assertLessThanOrEqual( 0, $results[$fields]['Time Patch - Trunk'], 'Improving performance should improve performance' );
+
 	}
 }