[wp-trac] [WordPress Trac] #23711: file include wrapper for media_sideload_image

WordPress Trac noreply at wordpress.org
Sat Mar 9 15:47:58 UTC 2013


#23711: file include wrapper for media_sideload_image
------------------------------------------------+--------------------------
 Reporter:  norcross                            |       Owner:
     Type:  enhancement                         |      Status:  new
 Priority:  normal                              |   Milestone:  Awaiting
Component:  Media                               |  Review
 Severity:  normal                              |     Version:  3.5
 Keywords:  has-patch dev-feedback 2nd-opinion  |  Resolution:
------------------------------------------------+--------------------------

Comment (by nacin):

 Also, the "proper" way to patch this using the include-only-what-is-needed
 suggestion would be to place these these require_once calls directly at
 the top of media_sideload_image(). It'd be weird to do that for only this
 function, though.

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


More information about the wp-trac mailing list