Changeset 31645 for trunk/src/wp-admin/includes/media.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/media.php (modified) (2 diffs)

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

@@ -2654 +2654 @@
         </div>
     <?php
-    elseif ( $attachment_id && 0 === strpos( $post->post_mime_type, 'audio/' ) ):
+    elseif ( $attachment_id && wp_attachment_is( 'audio', $post ) ):
 
         wp_maybe_generate_attachment_metadata( $post );
@@ -2660 +2660 @@
         echo wp_audio_shortcode( array( 'src' => $att_url ) );
 
-    elseif ( $attachment_id && 0 === strpos( $post->post_mime_type, 'video/' ) ):
+    elseif ( $attachment_id && wp_attachment_is( 'video', $post ) ):
 
         wp_maybe_generate_attachment_metadata( $post );