[wp-trac] [WordPress Trac] #61926: Imagick breaks default image progressive/interlace when converted to a mime type that doesn't support interlace

[WordPress Trac]

#61926: Imagick breaks default image progressive/interlace when converted to a mime
type that doesn't support interlace
-------------------------------+------------------------------
 Reporter:  kkmuffme           |       Owner:  (none)
     Type:  defect (bug)       |      Status:  new
 Priority:  normal             |   Milestone:  Awaiting Review
Component:  Media              |     Version:
 Severity:  normal             |  Resolution:
 Keywords:  reporter-feedback  |     Focuses:
-------------------------------+------------------------------

Comment (by kkmuffme):

 Replying to [comment:1 adamsilverstein]:
 > If you upload a JPEG interlaced and have `image_editor_output_format`
 set to transform that to WebP, I would expect a non interlaced output
 since WebP doesn't support interlace. Isn't that how it works currently?

 This class is not limited to uploads but can be used outside of it too.

 > > retrieveing it here
 https://github.com/WordPress/WordPress/blame/master/wp-includes/class-wp-
 image-editor-imagick.php#L827 is too late
 >
 > Are you sure that it is too late at that point, I would expect
 `$this->image` to be the original uploaded image, not the transformed mime
 type image here. Is that not the case?
 >
 > Can you please provide some steps to reproduce? what filters do you have
 in place? what type of image are you uploading? what are you expecting for
 output/what actually happens.

 Yes, it's not. The reason being it uses $this->image instance to set
 format instead of the passed param $image
 https://github.com/WordPress/WordPress/blame/master/wp-includes/class-wp-
 image-editor-imagick.php#L818

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/61926#comment:2>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform