Changeset 31058

Timestamp:

01/06/2015 04:57:12 PM (12 years ago)

Author:

SergeyBiryukov

Message:

Skip building the query in wp_count_posts() if cached results are used.

props MikeHansenMe.
fixes #30928.

File:

• trunk/src/wp-includes/post.php (modified) (2 diffs)

trunk/src/wp-includes/post.php

@@ -2352 +2352 @@
         $cache_key = _count_posts_cache_key( $type, $perm );
 
+        $counts = wp_cache_get( $cache_key, 'counts' );
+        if ( false !== $counts ) {
+                /** This filter is documented in wp-includes/post.php */
+                return apply_filters( 'wp_count_posts', $counts, $type, $perm );
+        }
+
         $query = "SELECT post_status, COUNT( * ) AS num_posts FROM {$wpdb->posts} WHERE post_type = %s";
         if ( 'readable' == $perm && is_user_logged_in() ) {
@@ -2363 +2369 @@
         $query .= ' GROUP BY post_status';
 
-        $counts = wp_cache_get( $cache_key, 'counts' );
-        if ( false === $counts ) {
-                $results = (array) $wpdb->get_results( $wpdb->prepare( $query, $type ), ARRAY_A );
-                $counts = array_fill_keys( get_post_stati(), 0 );
-
-                foreach ( $results as $row )
-                        $counts[ $row['post_status'] ] = $row['num_posts'];
-
-                $counts = (object) $counts;
-                wp_cache_set( $cache_key, $counts, 'counts' );
-        }
+        $results = (array) $wpdb->get_results( $wpdb->prepare( $query, $type ), ARRAY_A );
+        $counts = array_fill_keys( get_post_stati(), 0 );
+
+        foreach ( $results as $row ) {
+                $counts[ $row['post_status'] ] = $row['num_posts'];
+        }
+
+        $counts = (object) $counts;
+        wp_cache_set( $cache_key, $counts, 'counts' );
 
         /**