Changeset 31645 for trunk/src/wp-admin/includes/image.php

Timestamp:

03/06/2015 08:25:09 PM (10 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-admin/includes/image.php (modified) (1 diff)

trunk/src/wp-admin/includes/image.php

@@ -128 +128 @@
             $metadata['image_meta'] = $image_meta;
 
-    } elseif ( preg_match( '#^video/#', get_post_mime_type( $attachment ) ) ) {
+    } elseif ( wp_attachment_is( 'video', $attachment ) ) {
         $metadata = wp_read_video_metadata( $file );
         $support = current_theme_supports( 'post-thumbnails', 'attachment:video' ) || post_type_supports( 'attachment:video', 'thumbnail' );
-    } elseif ( preg_match( '#^audio/#', get_post_mime_type( $attachment ) ) ) {
+    } elseif ( wp_attachment_is( 'audio', $attachment ) ) {
         $metadata = wp_read_audio_metadata( $file );
         $support = current_theme_supports( 'post-thumbnails', 'attachment:audio' ) || post_type_supports( 'attachment:audio', 'thumbnail' );