Ticket #54476: 54476.4.diff

src/wp-includes/media.php

@@ -3942 +3942 @@
                 }
         }
 
+        $mime_type = isset( $args['mime_type'] ) ? $args['mime_type'] : wp_check_filetype( $path )['type'];
+
+        /** This filter is documented in wp-includes/class-wp-image-editor.php */
+        $output_format = apply_filters( 'image_editor_output_format', array(), $path, $mime_type );
+        if ( isset( $output_format[ $mime_type ] ) ) {
+                $args['output_mime_type'] = $output_format[ $mime_type ];
+        }
+
         $implementation = _wp_image_editor_choose( $args );
 
         if ( $implementation ) {
@@ -3994 +4002 @@
          *                                'WP_Image_Editor_Imagick', 'WP_Image_Editor_GD'.
          */
         $implementations = apply_filters( 'wp_image_editors', array( 'WP_Image_Editor_Imagick', 'WP_Image_Editor_GD' ) );
+        $supports_input  = false;
 
         foreach ( $implementations as $implementation ) {
                 if ( ! call_user_func( array( $implementation, 'test' ), $args ) ) {
                         continue;
                 }
 
+                // Implementation should support the passed mime type.
                 if ( isset( $args['mime_type'] ) &&
                         ! call_user_func(
                                 array( $implementation, 'supports_mime_type' ),
@@ -4008 +4018 @@
                         continue;
                 }
 
+                // Implementation should support requested methods.
                 if ( isset( $args['methods'] ) &&
                         array_diff( $args['methods'], get_class_methods( $implementation ) ) ) {
 
                         continue;
                 }
 
+                // Implementation should ideally support the output mime type as well if set and different than the passed type.
+                if (
+                        isset( $args['mime_type'] ) &&
+                        isset( $args['output_mime_type'] ) &&
+                        $args['mime_type'] !== $args['output_mime_type'] &&
+                        ! call_user_func( array( $implementation, 'supports_mime_type' ), $args['output_mime_type'] )
+                ) {
+                        // This implementation supports the imput type but not the output type.
+                        // Keep looking to see if we can find an implementation that supports both.
+                        $supports_input = $implementation;
+                        continue;
+                }
+
+                // Favor the implementation that supports both input and output mime types.
                 return $implementation;
         }
 
-        return false;
+        return $supports_input;
 }
 
 /**

tests/phpunit/tests/image/editor.php

@@ -126 +126 @@
                 $this->assertSame( 82, $editor->get_quality(), 'Default quality setting is 82.' );
 
                 // Quality should change to the output format's value.
-                // A PNG image will be converted to WEBP whose quialty should be 86.
+                // A PNG image will be converted to WEBP whose quality should be 86.
                 $editor->save();
                 $this->assertSame( 86, $editor->get_quality(), 'Output image format is WEBP. Quality setting for it should be 86.' );