[wp-trac] [WordPress Trac] #21183: Update SimplePie to 1.3

WordPress Trac wp-trac at lists.automattic.com
Tue Aug 28 14:05:29 UTC 2012


#21183: Update SimplePie to 1.3
--------------------------------+--------------------
 Reporter:  rmccue              |       Owner:  nacin
     Type:  defect (bug)        |      Status:  new
 Priority:  normal              |   Milestone:  3.5
Component:  External Libraries  |     Version:
 Severity:  normal              |  Resolution:
 Keywords:                      |
--------------------------------+--------------------

Comment (by rmccue):

 Two options with that: WP could just use the autoloader to do that, or
 include them all from one master file.

 Now that I think of it though, a better option would be a hybrid: include
 the classes that are almost always used, and let the autoloader handle the
 rest. That would mean: include `SimplePie`, `SimplePie_Item`,
 `SimplePie_Registry`, `SimplePie_IRI`, `SimplePie_Content_Type_Sniffer`,
 `SimplePie_Decode_HTML_Entities`, `SimplePie_gzdecode`,
 `SimplePie_Parse_Date` and `SimplePie_XML_Declaration_Parser`. The rest
 are either overriden by WordPress, or are accessor classes that are rarely
 used, and that case could be handled by the autoloader.

 What do you think?

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


More information about the wp-trac mailing list