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

Timestamp:

09/17/2015 05:02:52 AM (9 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 );