WordPress.org

Make WordPress Core

Opened 2 years ago

Closed 2 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:
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)

comment:1 @aaroncampbell2 years ago

  • Description modified (diff)

comment:2 @markjaquith2 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().

comment:3 @markjaquith2 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.