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

WordPress Trac noreply at wordpress.org
Fri Mar 6 17:50:05 UTC 2015


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

Comment (by DrewAPicture):

 Replying to [comment:13 wonderboymusic]:
 > @DrewAPicture - the internals need different functions to supply the
 list of accepted mime types - that new func might be useful for other
 types, but I don't see `pdf` as being useful now, since there are no
 instances in core that would use it.

 Regardless, as the proposed functions are essentially 97 percent the same,
 I'd still argue for a single function with a switch in it, even if we only
 cover the audio and video cases initially. It's the DRY-ght thing to do
 :-)

--
Ticket URL: <https://core.trac.wordpress.org/ticket/25275#comment:14>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list