Ticket #11168: 11168.diff

wp-includes/default-widgets.php

@@ -772 +772 @@
         if ( is_wp_error($rss) ) {
                 if ( is_admin() || current_user_can('manage_options') )
                         echo '<p>' . sprintf( __('<strong>RSS Error</strong>: %s'), $rss->get_error_message() ) . '</p>';
-                $rss->__destruct(); 
-                unset($rss);
                 return;
         }
 
@@ -953 +951 @@
                         $link = esc_url(strip_tags($rss->get_permalink()));
                         while ( stristr($link, 'http') != $link )
                                 $link = substr($link, 1);
+
+                        $rss->__destruct();
+                        unset($rss);
                 }
-                $rss->__destruct(); 
-                unset($rss);
         }
 
         return compact( 'title', 'url', 'link', 'items', 'error', 'show_summary', 'show_author', 'show_date' );

wp-admin/includes/dashboard.php

@@ -624 +624 @@
                         printf(__('<strong>RSS Error</strong>: %s'), $rss->get_error_message());
                         echo '</p>';
                 }
-                $rss->__destruct(); 
-                unset($rss);
                 return;
         }
 
@@ -742 +740 @@
                         printf(__('<strong>RSS Error</strong>: %s'), $rss->get_error_message());
                         echo '</p></div>';
                 }
-                $rss->__destruct(); 
-                unset($rss);
         } elseif ( !$rss->get_item_quantity() ) {
                 $rss->__destruct(); 
                 unset($rss);
@@ -932 +928 @@
                 // title is optional.  If black, fill it if possible
                 if ( !$widget_options[$widget_id]['title'] && isset($_POST['widget-rss'][$number]['title']) ) {
                         $rss = fetch_feed($widget_options[$widget_id]['url']);
-                        if ( ! is_wp_error($rss) )
-                                $widget_options[$widget_id]['title'] = htmlentities(strip_tags($rss->get_title()));
-                        else
+                        if ( is_wp_error($rss) ) {
                                 $widget_options[$widget_id]['title'] = htmlentities(__('Unknown Feed'));
-                        $rss->__destruct(); 
-                        unset($rss);
+                        } else {
+                                $widget_options[$widget_id]['title'] = htmlentities(strip_tags($rss->get_title()));     
+                                $rss->__destruct();
+                                unset($rss);                            
+                        }
                 }
                 update_option( 'dashboard_widget_options', $widget_options );
         }