Changeset 31645 for trunk/src/wp-includes/media.php

Timestamp:

03/06/2015 08:25:09 PM (11 years ago)

Author:

wonderboymusic

Message:

Introduce a function, wp_attachment_is( $type, $post = 0 ), to collapse the logic for determining whether an attachment is an image, audio, or video.

This is admittedly a first pass. There needs to be a generic handler for when any other type is passed, but for now it accepts the whitelist.

See #25275.

File:

• trunk/src/wp-includes/media.php (modified) (1 diff)

trunk/src/wp-includes/media.php

@@ -2977 +2977 @@
         $thumbnail_support = current_theme_supports( 'post-thumbnails', $post->post_type ) && post_type_supports( $post->post_type, 'thumbnail' );
         if ( ! $thumbnail_support && 'attachment' === $post->post_type && $post->post_mime_type ) {
-            if ( 0 === strpos( $post->post_mime_type, 'audio/' ) ) {
+            if ( wp_attachment_is( 'audio', $post ) ) {
                 $thumbnail_support = post_type_supports( 'attachment:audio', 'thumbnail' ) || current_theme_supports( 'post-thumbnails', 'attachment:audio' );
-            } elseif ( 0 === strpos( $post->post_mime_type, 'video/' ) ) {
+            } elseif ( wp_attachment_is( 'video', $post ) ) {
                 $thumbnail_support = post_type_supports( 'attachment:video', 'thumbnail' ) || current_theme_supports( 'post-thumbnails', 'attachment:video' );
             }