Make WordPress Core

Opened 13 years ago

Closed 13 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 Owned by:
Priority: normal Milestone:
Component: Media Version:
Severity: normal Keywords: needs-patch
Cc: Focuses:

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
13 years ago

  • Description modified (diff)

#2 @markjaquith
13 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
13 years ago

  • Milestone 3.6
  • Resolution → invalid
  • Status new → 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.