[wp-trac] [WordPress Trac] #8964: Allow adding headers to get_plugin_data

WordPress Trac wp-trac at lists.automattic.com
Thu Aug 27 20:59:21 UTC 2009


#8964: Allow adding headers to get_plugin_data
-------------------------+--------------------------------------------------
 Reporter:  strider72    |       Owner:  westi                        
     Type:  enhancement  |      Status:  accepted                     
 Priority:  normal       |   Milestone:  2.9                          
Component:  Plugins      |     Version:                               
 Severity:  normal       |    Keywords:  has-patch tested commit early
-------------------------+--------------------------------------------------

Comment(by westi):

 Ok had a look over this.

 I'm wondering if the following changes might improve things:

  * Rename the {{{plugin_headers}}} filter to {{{extra_plugin_headers}}}
  * Expect the array back from {{{extra_plugin_headers}}} filter which
 contains Name => Regex match like the one specified for the normal
 headers.
  * Use array_merge($extras,$defaults) so that you can't overwrite the
 defaults.

 Otherwise all looks good :-)

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


More information about the wp-trac mailing list