[wp-trac] [WordPress Trac] #25275: Checking if attachment is audio/video

WordPress Trac noreply at wordpress.org
Thu Sep 12 20:56:57 UTC 2013


#25275: Checking if attachment is audio/video
--------------------------+------------------------------
 Reporter:  danielpataki  |       Owner:
     Type:  enhancement   |      Status:  new
 Priority:  normal        |   Milestone:  Awaiting Review
Component:  Media         |     Version:  3.6
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |
--------------------------+------------------------------

Comment (by danielpataki):

 Agreed on all counts! I will modify the code to reflect al that.

 I'll add a wp_attachment_is_type() function which uses wp_ext2type()
 internally.

 Sorry for being a bit of a dunce but I'm completely new to making patches.
 When you say "generate patches from the WordPress root", does this mean to
 diff the whole directory? I basically used: diff -u post.php post-mine.php
 > post.diff to create the diff file. Would the method to generate from the
 root be to have a vanilla copy of wordpress and another one where I make
 changes side by side and then just use this: diff -u wordpress wordpress-
 mine > wordpress-diff.

 Also, since wp_ext2type is only in alpha, do you recommend always creating
 patches from an SVN checkout of the trunk?

 Sorry for all the questions!

 Daniel

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


More information about the wp-trac mailing list