WordPress.org

Make WordPress Core

Ticket #9198: simplepie.diff

File simplepie.diff, 8.0 KB (added by ryan, 5 years ago)

Switch widgets over to simplepie. Caching not done yet. Assumes simplepie.inc in wp-includes

  • wp-includes/feed.php

     
    534534        return apply_filters( 'feed_content_type', $content_type, $type ); 
    535535} 
    536536 
     537function fetch_feed($url) { 
     538        require_once (ABSPATH . WPINC . '/simplepie.inc'); 
     539 
     540        $feed = new SimplePie(); 
     541        $feed->set_feed_url($url); 
     542        $feed->enable_cache(false); // TODO Create custom cache class that uses get_transient() and set_transient(). 
     543        $feed->init(); 
     544        $feed->handle_content_type(); 
     545 
     546        return $feed; 
     547} 
     548 
    537549?> 

  • wp-includes/widgets.php

     
    15011501        if ( empty($url) ) 
    15021502                return; 
    15031503 
    1504         require_once(ABSPATH . WPINC . '/rss.php'); 
    1505  
    1506         $rss = fetch_rss($url); 
    1507         $link = clean_url(strip_tags($rss->channel['link'])); 
     1504        $rss = fetch_feed($url); 
     1505        $link = clean_url(strip_tags($rss->get_permalink())); 
    15081506        while ( strstr($link, 'http') != $link ) 
    15091507                $link = substr($link, 1); 
    1510         $desc = attribute_escape(strip_tags(html_entity_decode($rss->channel['description'], ENT_QUOTES))); 
     1508        $desc = attribute_escape(strip_tags(html_entity_decode($rss->get_description(), ENT_QUOTES))); 
    15111509        $title = $options[$number]['title']; 
    15121510        if ( empty($title) ) 
    1513                 $title = htmlentities(strip_tags($rss->channel['title'])); 
     1511                $title = htmlentities(strip_tags($rss->get_title())); 
    15141512        if ( empty($title) ) 
    15151513                $title = $desc; 
    15161514        if ( empty($title) ) 
    … …  
    15411539 */ 
    15421540function wp_widget_rss_output( $rss, $args = array() ) { 
    15431541        if ( is_string( $rss ) ) { 
    1544                 require_once(ABSPATH . WPINC . '/rss.php'); 
    1545                 if ( !$rss = fetch_rss($rss) ) 
     1542                if ( !$rss = fetch_feed($rss) ) 
    15461543                        return; 
    15471544        } elseif ( is_array($rss) && isset($rss['url']) ) { 
    1548                 require_once(ABSPATH . WPINC . '/rss.php'); 
    15491545                $args = $rss; 
    1550                 if ( !$rss = fetch_rss($rss['url']) ) 
     1546                if ( !$rss = fetch_feed($rss['url']) ) 
    15511547                        return; 
    15521548        } elseif ( !is_object($rss) ) { 
    15531549                return; 
    … …  
    15641560        $show_author   = (int) $show_author; 
    15651561        $show_date     = (int) $show_date; 
    15661562 
    1567         if ( is_array( $rss->items ) && !empty( $rss->items ) ) { 
    1568                 $rss->items = array_slice($rss->items, 0, $items); 
    1569                 echo '<ul>'; 
    1570                 foreach ( (array) $rss->items as $item ) { 
    1571                         while ( strstr($item['link'], 'http') != $item['link'] ) 
    1572                                 $item['link'] = substr($item['link'], 1); 
    1573                         $link = clean_url(strip_tags($item['link'])); 
    1574                         $title = attribute_escape(strip_tags($item['title'])); 
    1575                         if ( empty($title) ) 
    1576                                 $title = __('Untitled'); 
    1577                         $desc = ''; 
    1578                         if ( isset( $item['description'] ) && is_string( $item['description'] ) ) 
    1579                                 $desc = str_replace(array("\n", "\r"), ' ', attribute_escape(strip_tags(html_entity_decode($item['description'], ENT_QUOTES)))); 
    1580                         elseif ( isset( $item['summary'] ) && is_string( $item['summary'] ) ) 
    1581                                 $desc = str_replace(array("\n", "\r"), ' ', attribute_escape(strip_tags(html_entity_decode($item['summary'], ENT_QUOTES)))); 
    1582                         if ( 360 < strlen( $desc ) ) 
    1583                                 $desc = wp_html_excerpt( $desc, 360 ) . ' [&hellip;]'; 
    1584                         $summary = $desc; 
     1563        if ( !$rss->get_item_quantity() ) { 
     1564                echo '<ul><li>' . __( 'An error has occurred; the feed is probably down. Try again later.' ) . '</li></ul>'; 
     1565                return; 
     1566        } 
    15851567 
    1586                         if ( $show_summary ) { 
    1587                                 $desc = ''; 
    1588                                 $summary = wp_specialchars( $summary ); 
    1589                                 $summary = "<div class='rssSummary'>$summary</div>"; 
    1590                         } else { 
    1591                                 $summary = ''; 
    1592                         } 
     1568        echo '<ul>'; 
     1569        $count = 0; 
     1570        foreach ( $rss->get_items() as $item ) { 
     1571                $link = $item->get_link(); 
     1572                while ( strstr($link, 'http') != $link ) 
     1573                        $link = substr($link, 1); 
     1574                $link = clean_url(strip_tags($link)); 
     1575                $title = attribute_escape(strip_tags($item->get_title())); 
     1576                if ( empty($title) ) 
     1577                        $title = __('Untitled'); 
    15931578 
    1594                         $date = ''; 
    1595                         if ( $show_date ) { 
    1596                                 if ( isset($item['pubdate']) ) 
    1597                                         $date = $item['pubdate']; 
    1598                                 elseif ( isset($item['published']) ) 
    1599                                         $date = $item['published']; 
     1579                $desc = str_replace(array("\n", "\r"), ' ', attribute_escape(strip_tags(html_entity_decode($item->get_description(), ENT_QUOTES)))); 
     1580                $desc = wp_html_excerpt( $desc, 360 ) . ' [&hellip;]'; 
     1581                $desc = wp_specialchars( $desc ); 
    16001582 
    1601                                 if ( $date ) { 
    1602                                         if ( $date_stamp = strtotime( $date ) ) 
    1603                                                 $date = ' <span class="rss-date">' . date_i18n( get_option( 'date_format' ), $date_stamp ) . '</span>'; 
    1604                                         else 
    1605                                                 $date = ''; 
    1606                                 } 
    1607                         } 
     1583                if ( $show_summary ) { 
     1584                        $summary = "<div class='rssSummary'>$desc</div>"; 
     1585                } else { 
     1586                        $summary = ''; 
     1587                } 
    16081588 
    1609                         $author = ''; 
    1610                         if ( $show_author ) { 
    1611                                 if ( isset($item['dc']['creator']) ) 
    1612                                         $author = ' <cite>' . wp_specialchars( strip_tags( $item['dc']['creator'] ) ) . '</cite>'; 
    1613                                 elseif ( isset($item['author_name']) ) 
    1614                                         $author = ' <cite>' . wp_specialchars( strip_tags( $item['author_name'] ) ) . '</cite>'; 
    1615                         } 
     1589                $date = ''; 
     1590                if ( $show_date ) { 
     1591                        $date = $item->get_date(); 
    16161592 
    1617                         if ( $link == '' ) { 
    1618                                 echo "<li>$title{$date}{$summary}{$author}</li>"; 
    1619                         } else { 
    1620                                 echo "<li><a class='rsswidget' href='$link' title='$desc'>$title</a>{$date}{$summary}{$author}</li>"; 
     1593                        if ( $date ) { 
     1594                                if ( $date_stamp = strtotime( $date ) ) 
     1595                                        $date = ' <span class="rss-date">' . date_i18n( get_option( 'date_format' ), $date_stamp ) . '</span>'; 
     1596                                else 
     1597                                        $date = ''; 
    16211598                        } 
    1622 } 
    1623                 echo '</ul>'; 
    1624         } else { 
    1625                 echo '<ul><li>' . __( 'An error has occurred; the feed is probably down. Try again later.' ) . '</li></ul>'; 
     1599                } 
     1600 
     1601                $author = ''; 
     1602                if ( $show_author ) { 
     1603                        $author = $item->get_author(); 
     1604                        $author = $author->get_name(); 
     1605                        $author = ' <cite>' . wp_specialchars( strip_tags( $author ) ) . '</cite>'; 
     1606                } 
     1607 
     1608                if ( $link == '' ) { 
     1609                        echo "<li>$title{$date}{$summary}{$author}</li>"; 
     1610                } else { 
     1611                        echo "<li><a class='rsswidget' href='$link' title='$desc'>$title</a>{$date}{$summary}{$author}</li>"; 
     1612                } 
     1613                $count++; 
     1614                if ( $count >= $items ) 
     1615                        break; 
    16261616        } 
     1617        echo '</ul>'; 
    16271618} 
    16281619 
    16291620/** 
    … …  
    18101801        $show_date     = (int) $widget_rss['show_date']; 
    18111802 
    18121803        if ( $check_feed ) { 
    1813                 require_once(ABSPATH . WPINC . '/rss.php'); 
    1814                 $rss = fetch_rss($url); 
     1804                $rss = fetch_feed($url); 
    18151805                $error = false; 
    18161806                $link = ''; 
    18171807                if ( !is_object($rss) ) { 
    18181808                        $url = wp_specialchars(__('Error: could not find an RSS or ATOM feed at that URL.'), 1); 
    18191809                        $error = sprintf(__('Error in RSS %1$d'), $widget_number ); 
    18201810                } else { 
    1821                         $link = clean_url(strip_tags($rss->channel['link'])); 
     1811                        $link = clean_url(strip_tags($rss->get_permalink())); 
    18221812                        while ( strstr($link, 'http') != $link ) 
    18231813                                $link = substr($link, 1); 
    18241814                } 

  • wp-admin/includes/dashboard.php

     
    670670function wp_dashboard_secondary_output() { 
    671671        $widgets = get_option( 'dashboard_widget_options' ); 
    672672        @extract( @$widgets['dashboard_secondary'], EXTR_SKIP ); 
    673         $rss = @fetch_rss( $url ); 
     673        $rss = @fetch_feed( $url ); 
    674674 
    675         if ( !isset($rss->items) || 0 == count($rss->items) ) 
     675        if ( !$rss->get_item_quantity() ) 
    676676                return false; 
    677677 
    678         $rss->items = array_slice($rss->items, 0, $items); 
    679  
    680         if ( 'http://planet.wordpress.org/' == $rss->channel['link'] ) { 
    681                 foreach ( array_keys($rss->items) as $i ) { 
    682                         list($site, $description) = explode( ':', wp_specialchars($rss->items[$i]['title']), 2 ); 
    683                         $rss->items[$i]['dc']['creator'] = trim($site); 
    684                         $rss->items[$i]['title'] = trim($description); 
    685                 } 
    686         } 
    687  
    688678        echo "<div class='rss-widget'>"; 
    689679        wp_widget_rss_output( $rss, $widgets['dashboard_secondary'] ); 
    690680        echo "</div>"; 
    … …  
    778768        } 
    779769 
    780770 
    781         require_once( ABSPATH . WPINC . '/rss.php' ); 
    782         init(); // initialize rss constants 
    783  
    784         $cache = new RSSCache( MAGPIE_CACHE_DIR, MAGPIE_CACHE_AGE ); 
    785  
     771        /* TODO Cache check here. 
    786772        foreach ( $check_urls as $check_url ) { 
    787                 $status = $cache->check_cache( $check_url ); 
     773 
    788774                if ( 'HIT' !== $status ) { 
    789775                        echo $loading; 
    790776                        return false; 
    791777                } 
    792778        } 
     779        */ 
    793780 
    794781        if ( $callback && is_callable( $callback ) ) { 
    795782                $args = array_slice( func_get_args(), 2 );