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

Timestamp:

09/17/2015 05:02:52 AM (11 years ago)

Author:

wonderboymusic

Message:

Media: In wp_prepare_attachment_for_js(), don't call file_exists() and filesize() to retrieve $bytes if the data is already present in $meta. This is how the same code in attachment_submitbox_metadata() already works.

Props polevaultweb.
Fixes #33214.

File:

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

trunk/src/wp-includes/media.php

@@ -2779 +2779 @@
 
         $attached_file = get_attached_file( $attachment->ID );
-        if ( file_exists( $attached_file ) ) {
+        
+        if ( isset( $meta['filesize'] ) ) {
+                $bytes = $meta['filesize'];
+        } elseif ( file_exists( $attached_file ) ) {
                 $bytes = filesize( $attached_file );
+        } else {
+                $bytes = '';
+        }
+
+        if ( $bytes ) {
                 $response['filesizeInBytes'] = $bytes;
                 $response['filesizeHumanReadable'] = size_format( $bytes );