[wp-trac] [WordPress Trac] #29204: SimplePie error with fetch_feed()

WordPress Trac noreply at wordpress.org
Tue Dec 20 22:48:18 UTC 2016


#29204: SimplePie error with fetch_feed()
-----------------------------------------------+---------------------------
 Reporter:  useStrict                          |       Owner:
     Type:  defect (bug)                       |  SergeyBiryukov
 Priority:  normal                             |      Status:  reviewing
Component:  Feeds                              |   Milestone:  Awaiting
 Severity:  normal                             |  Review
 Keywords:  has-patch 4.7-early needs-refresh  |     Version:  3.8
                                               |  Resolution:
                                               |     Focuses:
-----------------------------------------------+---------------------------

Comment (by thefarlilacfield):

 I'm writing to confirm that `29204.diff` fixes this.

 For the sake of reference (search results) it's worth noting a few things.

 This change removes WP_Feed_Cache as a dependency of the WordPress core.

 The `set_cache_class()` method that is being removed by the patch had
 already been depreciated as of the 2012 version of SimplePie (1.3.1)
 relied upon by WordPress (up to 4.7 at least).

 My look at the code has been cursory, but it appears that WordPress is
 relying upon legacy features of the library that inherently require a
 dynamic method to be called as static.  As of PHP 5 this has merited an an
 `E_STRICT` warning.  PHP 7 escalates this to an `E_DEPRECATED` warning,
 and the current documentation notes that support may be removed in the
 future.

 It also appears that the maintainers of SimplePie have specifically
 assumed the legacy burden of WordPress support all the way up to version
 1.4.3, maintaining backwards compatibility
 [https://github.com/simplepie/simplepie/issues/443#issuecomment-222282500
 ""unless there's a really good reason not to""]
 ([https://github.com/mblaney mblaney] being is the current maintainer),
 and that this consideration has overshadowed discussion of a 2.0.0
 revision of their library.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/29204#comment:19>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list