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

WordPress Trac noreply at wordpress.org
Fri Mar 8 15:01:38 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 norcross):

 the problem is that to use `media_sideload_image` on the front end, these
 files have to be included. so any theme / plugin that makes use of this is
 suspect to changes in core. by creating this wrapper (and Sergey is
 correct with the `ABSPATH` part, not sure why it worked on my local) it
 allows core to make changes down the road without breaking existing sites.

 also, I don't see it as a hack. if anything, the current method (having to
 manually call those files) is more of a hack.

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


More information about the wp-trac mailing list