WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#24504 closed defect (bug) (invalid)

Have wp_get_(audio|video)_extensions() wrap wp_get_mime_types( $type = null ) if feasible

Reported by: aaroncampbell's profile aaroncampbell Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Media Keywords: needs-patch
Focuses: Cc:

Description (last modified by aaroncampbell)

Have wp_get_(audio|video)_extensions() wrap wp_get_mime_types( $type = null ) if feasible

As per ​http://make.wordpress.org/core/2013/05/29/post-formats-ui-is-exiting-core-will-live-as-a-plugin/

Change History (3)

#1 @aaroncampbell
12 years ago

  • Description modified (diff)

#2 @markjaquith
12 years ago

Strategy: pass in a "before the slash" part of the mime type to wp_get_mime_types() which will only return those ones. Get rid of wp_get_audio_extensions() and wp_get_video_extensions().

#3 @markjaquith
12 years ago

  • Milestone 3.6 deleted
  • Resolution set to invalid
  • Status changed from new to closed

So, ME.js only supports a subset of the video/audio MIME types WP knows about. So allowing all of them would lead to embeds being processed that won't work. Not great. We should keep these separate as they are for different purposes.

List of supported MIME types here:

​https://github.com/johndyer/mediaelement/blob/master/src/js/me-namespace.js

Note: See TracTickets for help on using tickets.