Ticket #54476: 54476.6.diff

src/wp-includes/media.php

@@ -3838 +3838 @@
 function wp_get_image_editor( $path, $args = array() ) {
         $args['path'] = $path;
 
+        // If the mime type is not set in args, try to extract and set it from the file.
         if ( ! isset( $args['mime_type'] ) ) {
                 $file_info = wp_check_filetype( $args['path'] );
 
@@ -3848 +3849 @@
                 }
         }
 
+        // Check and set the output mime type mapped to the input type.
+        if ( isset( $args['mime_type'] ) ) {
+                /** This filter is documented in wp-includes/class-wp-image-editor.php */
+                $output_format = apply_filters( 'image_editor_output_format', array(), $path, $args['mime_type'] );
+                if ( isset( $output_format[ $args['mime_type'] ] ) ) {
+                        $args['output_mime_type'] = $output_format[ $args['mime_type'] ];
+                }
+        }
+
         $implementation = _wp_image_editor_choose( $args );
 
         if ( $implementation ) {
@@ -3900 +3910 @@
          *                                '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' ),
@@ -3914 +3926 @@
                         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;
 }
 
 /**