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

WordPress Trac wp-trac at lists.automattic.com
Fri Aug 3 05:19:50 GMT 2007

#4547: Convert WP from magpie to Simplepie
 Reporter:  technosailor          |        Owner:  technosailor
     Type:  enhancement           |       Status:  closed      
 Priority:  normal                |    Milestone:              
Component:  General               |      Version:  2.3         
 Severity:  normal                |   Resolution:  wontfix     
 Keywords:  simplepie magpie rss  |  
Comment (by Otto42):

 Replying to [comment:26 markjaquith]:
 > Also, I think your patch has it backwards!

 Argh. I swear that there was a ! there when I wrote it!

 Replying to [comment:28 markjaquith]:
 > Although I intentionally prevented the hook from replacing the Magpie
 class, I'm open to arguments for why it should be able to do so.  The best
 argument that comes to mind is a reduction of the amount of PHP code that
 has to be parsed... is there another?

 Seems like enough to me. There's a huge amount of code there that is
 essentially useless if you're writing a plugin to replace it. All of the
 rss.php, all of the Snoopy stuff, etc. I mean, with the current codebase
 (2.2.1) I dropped my replacement code over the file entirely instead of
 using a plugin method, if you're replacing those functions, there's no
 reason to keep the old stuff.

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

More information about the wp-trac mailing list