[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