Changeset 59317

Timestamp:

10/29/2024 05:59:10 AM (22 months ago)

Author:

azaozz

Message:

Media: Fix converting of all HEIC/HEIF images to JPEGs after uploading regardless of dimensions.

Props ironprogrammer, adamsilverstein, azaozz.
Fixes #62305.

Location:

trunk/src

Files:

• wp-admin/includes/image.php (modified) (4 diffs)
• wp-includes/media.php (modified) (2 diffs)

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

@@ -292 +292 @@
                  * scale the image and use it as the "full" size.
                  */
+                $scale_down = false;
+                $convert    = false;
+
                 if ( $threshold && ( $image_meta['width'] > $threshold || $image_meta['height'] > $threshold ) ) {
+                        // The image will be converted if needed on saving.
+                        $scale_down = true;
+                } else {
+                        // The image may need to be converted regardless of its dimensions.
+                        $output_format = wp_get_image_editor_output_format( $file, $imagesize['mime'] );
+
+                        if ( is_array( $output_format ) && array_key_exists( $imagesize['mime'], $output_format ) ) {
+                                $convert = true;
+                        }
+                }
+
+                if ( $scale_down || $convert ) {
                         $editor = wp_get_image_editor( $file );
 
@@ -300 +315 @@
                         }
 
-                        // Resize the image.
-                        $resized = $editor->resize( $threshold, $threshold );
+                        if ( $scale_down ) {
+                                // Resize the image. This will also convet it if needed.
+                                $resized = $editor->resize( $threshold, $threshold );
+                        } elseif ( $convert ) {
+                                // The image will be converted (if possible) when saved.
+                                $resized = true;
+                        }
+
                         $rotated = null;
 
@@ -307 +328 @@
                         if ( ! is_wp_error( $resized ) && is_array( $exif_meta ) ) {
                                 $resized = $editor->maybe_exif_rotate();
-                                $rotated = $resized;
+                                $rotated = $resized; // bool true or WP_Error
                         }
 
@@ -315 +336 @@
                                  * This doesn't affect the sub-sizes names as they are generated from the original image (for best quality).
                                  */
-                                $saved = $editor->save( $editor->generate_filename( 'scaled' ) );
+                                if ( $scale_down ) {
+                                        $saved = $editor->save( $editor->generate_filename( 'scaled' ) );
+                                } else {
+                                        $saved = $editor->save();
+                                }
 
                                 if ( ! is_wp_error( $saved ) ) {

trunk/src/wp-includes/media.php

@@ -6234 +6234 @@
  */
 function wp_get_image_editor_output_format( $filename, $mime_type ) {
+        $default_output_format = array(
+                'image/heic'          => 'image/jpeg',
+                'image/heif'          => 'image/jpeg',
+                'image/heic-sequence' => 'image/jpeg',
+                'image/heif-sequence' => 'image/jpeg',
+        );
+
         /**
          * Filters the image editor output format mapping.
          *
-         * Enables filtering the mime type used to save images. By default,
-         * the mapping array is empty, so the mime type matches the source image.
+         * Enables filtering the mime type used to save images. By default HEIC/HEIF images
+         * are converted to JPEGs.
          *
          * @see WP_Image_Editor::get_output_format()
          *
          * @since 5.8.0
-         * @since 6.7.0 The default was changed from array() to array( 'image/heic' => 'image/jpeg' ).
+         * @since 6.7.0 The default was changed from empty array to array containing the HEIC mime types.
          *
          * @param string[] $output_format {
@@ -6254 +6261 @@
          * @param string $mime_type The source image mime type.
          */
-        return apply_filters( 'image_editor_output_format', array( 'image/heic' => 'image/jpeg' ), $filename, $mime_type );
-}
+        return apply_filters( 'image_editor_output_format', $default_output_format, $filename, $mime_type );
+}