[wp-trac] Re: [WordPress Trac] #4547: Convert WP from magpie to Simplepie

[WordPress Trac]

#4547: Convert WP from magpie to Simplepie
----------------------------------+-----------------------------------------
 Reporter:  technosailor          |        Owner:  technosailor
     Type:  defect                |       Status:  new         
 Priority:  normal                |    Milestone:  2.3 (trunk) 
Component:  General               |      Version:  2.3         
 Severity:  normal                |   Resolution:              
 Keywords:  simplepie magpie rss  |  
----------------------------------+-----------------------------------------
Comment (by matt):

 Load weight isn't a big deal, as you said it's only loaded when needed.
 I'm more concerned with the weight of maintaining and securing so much new
 code, related to the comment I left on your blog.

 As an alternative I see either maintaining the status quo with the package
 we currently use, which parses the feeds we use Good Enough, or creating
 an even more simplistic parser designed just to work with our dashboard
 feeds. (It could also be a lot more memory/space efficient if it didn't
 store the whole feed.)

 I don't buy the argument of parsing more types of feeds than we currently
 do as enough to justify the change, because even SimplePie, which is way
 better than Magpie from what I can see, is still pretty limited when
 compared to what Google does or UFP, particularly with regard to malformed
 or invalid feeds.

 If parsing everything possible is our goal, we should probably use a web
 service like Google's feed API, not switch out one almost-there library
 for another.

-- 
Ticket URL: <http://trac.wordpress.org/ticket/4547#comment:12>
WordPress Trac <http://trac.wordpress.org/>
WordPress blogging software