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

Timestamp:

10/09/2024 11:30:05 PM (2 years ago)

Author:

peterwilsoncc

Message:

Media: Update file size meta data when editing images.

Fixes an error in which the file size meta data retained the original upload's values follow a user editing the images in the media screen.

The original images' file sizes are now stored in the backup image data to allow for them to be restored when a user restores the original image.

Props ankit-k-gupta, antpb, audrasjb, chaion07, gauravsingh7, joedolson, oglekler, pls78, rajinsharwar, sayedulsayem, vertisoft.
Fixes #59684.

File:

• trunk/src/wp-admin/includes/image-edit.php (modified) (5 diffs)

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

@@ -827 +827 @@
                         } elseif ( isset( $meta['width'], $meta['height'] ) ) {
                                 $backup_sizes[ "full-$suffix" ] = array(
-                                        'width'  => $meta['width'],
-                                        'height' => $meta['height'],
-                                        'file'   => $parts['basename'],
+                                        'width'    => $meta['width'],
+                                        'height'   => $meta['height'],
+                                        'filesize' => $meta['filesize'],
+                                        'file'     => $parts['basename'],
                                 );
                         }
@@ -840 +841 @@
                 $meta['width']  = $data['width'];
                 $meta['height'] = $data['height'];
+                if ( isset( $data['filesize'] ) ) {
+                        /*
+                         * Restore the original filesize if it was backed up.
+                         *
+                         * See https://core.trac.wordpress.org/ticket/59684.
+                         */
+                        $meta['filesize'] = $data['filesize'];
+                }
         }
 
@@ -998 +1007 @@
         }
 
+        $saved_image = wp_save_image_file( $new_path, $img, $post->post_mime_type, $post_id );
         // Save the full-size file, also needed to create sub-sizes.
-        if ( ! wp_save_image_file( $new_path, $img, $post->post_mime_type, $post_id ) ) {
+        if ( ! $saved_image ) {
                 $return->error = esc_js( __( 'Unable to save the image.' ) );
                 return $return;
@@ -1019 +1029 @@
                 if ( $tag ) {
                         $backup_sizes[ $tag ] = array(
-                                'width'  => $meta['width'],
-                                'height' => $meta['height'],
-                                'file'   => $basename,
+                                'width'    => $meta['width'],
+                                'height'   => $meta['height'],
+                                'filesize' => $meta['filesize'],
+                                'file'     => $basename,
                         );
                 }
@@ -1029 +1040 @@
                 $meta['file'] = _wp_relative_upload_path( $new_path );
 
-                $size           = $img->get_size();
-                $meta['width']  = $size['width'];
-                $meta['height'] = $size['height'];
+                $size             = $img->get_size();
+                $meta['width']    = $size['width'];
+                $meta['height']   = $size['height'];
+                $meta['filesize'] = $saved_image['filesize'];
 
                 if ( $success && ( 'nothumb' === $target || 'all' === $target ) ) {