Changeset 18228

Timestamp:

06/10/2011 06:31:39 AM (14 years ago)

Author:

dd32

Message:

Cache the Dashboard RSS Widgets HTML output to reduce the memory footprint of including SimplePie. Fixes #16927

Location:

trunk/wp-admin

Files:

• includes/dashboard.php (modified) (3 diffs)
• index-extra.php (modified) (2 diffs)

trunk/wp-admin/includes/dashboard.php

@@ -1032 +1032 @@
 function wp_dashboard_cached_rss_widget( $widget_id, $callback, $check_urls = array() ) {
     $loading = '<p class="widget-loading hide-if-no-js">' . __( 'Loading&#8230;' ) . '</p><p class="hide-if-js">' . __( 'This widget requires JavaScript.' ) . '</p>';
+    $doing_ajax = ( defined('DOING_AJAX') && DOING_AJAX );
 
     if ( empty($check_urls) ) {
         $widgets = get_option( 'dashboard_widget_options' );
-        if ( empty($widgets[$widget_id]['url']) ) {
+        if ( empty($widgets[$widget_id]['url']) && ! $doing_ajax ) {
             echo $loading;
             return false;
@@ -1042 +1043 @@
     }
 
-    include_once ABSPATH . WPINC . '/class-feed.php';
-    foreach ( $check_urls as $check_url ) {
-        $cache = new WP_Feed_Cache_Transient('', md5($check_url), '');
-        if ( ! $cache->load() ) {
-            echo $loading;
-            return false;
-        }
+    $cache_key = 'dash_' . md5( $callback . implode(',', $check_urls) );
+    if ( false !== ( $output = get_transient( $cache_key ) ) ) {
+        echo $output;
+        return true;
+    }
+
+    if ( ! $doing_ajax ) {
+        echo $loading;
+        return false;
     }
 
@@ -1054 +1057 @@
         $args = array_slice( func_get_args(), 2 );
         array_unshift( $args, $widget_id );
+        ob_start();
         call_user_func_array( $callback, $args );
+        set_transient( $cache_key, ob_get_flush(), 43200); // Default lifetime in cache of 12 hours (same as the feeds)     
     }
 

trunk/wp-admin/index-extra.php

@@ -6 +6 @@
  * @subpackage Administration
  */
+
+define('DOING_AJAX', true);
 
 /** Load WordPress Bootstrap */
@@ -19 +21 @@
 
 case 'dashboard_incoming_links' :
-    wp_dashboard_incoming_links_output();
+    wp_dashboard_incoming_links();
     break;
 
 case 'dashboard_primary' :
-    wp_dashboard_rss_output( 'dashboard_primary' );
+    wp_dashboard_primary();
     break;
 
 case 'dashboard_secondary' :
-    wp_dashboard_secondary_output();
+    wp_dashboard_secondary();
     break;
 
 case 'dashboard_plugins' :
-    wp_dashboard_plugins_output();
+    wp_dashboard_plugins();
     break;