Changeset 31058

Timestamp:

01/06/2015 04:57:12 PM (10 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' );
 
     /**