Changeset 59317

Timestamp:

10/29/2024 05:59:10 AM (5 weeks 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 );
+}