[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