[wp-trac] [WordPress Trac] #23572: Add functions to parse Audio / Video data out of arbitrary content or a post
WordPress Trac
noreply at wordpress.org
Thu Mar 28 18:10:08 UTC 2013
#23572: Add functions to parse Audio / Video data out of arbitrary content or a
post
-------------------------------------------------+-------------------------
Reporter: wonderboymusic | Owner:
Type: enhancement | markjaquith
Priority: normal | Status: reopened
Component: Post Formats | Milestone: 3.6
Severity: normal | Version:
Keywords: needs-unit-tests has-patch commit | Resolution:
needs-codex |
-------------------------------------------------+-------------------------
Comment (by johnjamesjacoby):
Replying to [comment:46 greenshady]:
> The name of the plugin is "Get the Image". At the time I created the
plugin, using `get_the_image` made a lot of sense. If I had created the
plugin today, I'd probably name it different. Five years ago when the
plugin was created, I didn't know about prefixing stuff. It wasn't
something that was preached that much.
I understand, and assumed as much.
> One thing I didn't mention above is the the Get the Image plugin is
included with many themes and is used on millions of Web sites in that
manner. Themes can't overwrite pluggable functions. It's also used on
WordPress.com, which is problematic.
At the risk of sounding dramatic, it's another example of why including
plugins in themes is such a bad idea; functions.php ends up being a booby-
trap.
> I actually wish we'd do that with all WordPress functions, but that's a
different discussion. I say we should definitely do that for these
functions.
Seems like the most logical choice given the circumstance. A shame to
break the convention, since most other loop-oriented template tags aren't
prefixed.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/23572#comment:47>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list