[wp-trac] [WordPress Trac] #21017: Using force_feed() for simplepie in core

[WordPress Trac]

#21017: Using force_feed() for simplepie in core
------------------------------------+------------------------------
 Reporter:  josh401                 |       Owner:  nacin
     Type:  defect (bug)            |      Status:  reviewing
 Priority:  normal                  |   Milestone:  Awaiting Review
Component:  Feeds                   |     Version:  3.4
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+------------------------------
Changes (by devgav):

 * keywords:  has-patch => has-patch dev-feedback
 * owner:   => nacin
 * status:  new => reviewing


Comment:

 Just to clarify my views on '''force_feed()''' here ... I don't feel this
 is the solution.

 An invalid feed ''should'' fail by design. We don't want to force it
 through. force_Feed in SimplePie was designed to handle incorrect mime
 types, not to magically fixed broken feeds. These should be taken up with
 the feed provider.

 SimplePie Ref: http://simplepie.org/wiki/reference/simplepie/force_feed

 Hiding the error message, as per patch/Nacin's request should be the
 solution in this ticket.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/21017#comment:8>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software