[wp-trac] [WordPress Trac] #35973: Imagick basic mistakes & progressive

[WordPress Trac]

#35973: Imagick basic mistakes & progressive
--------------------------+-----------------------------
 Reporter:  DuckDagobert  |      Owner:
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  Awaiting Review
Component:  Media         |    Version:  4.4.2
 Severity:  normal        |   Keywords:
  Focuses:                |
--------------------------+-----------------------------
 Hey,

 I am using
 Array
 (
     [versionNumber] => 1655
     [versionString] => ImageMagick 6.7.7-10 2014-03-08 Q16
 http://www.imagemagick.org
 )

 in /wp-includes/class-wp-image-editor-imagick.php line 57-75 you list the
 required methods WP needs in imagick to later check using array_diff.
 The problem is that you use old capitalisation e.g. getimage but Imagick
 lately (since at least 1 year already) uses getImage, writeImage,... so
 this will cause that imagick is never used.

 Array
 (
     [3] => getimage
     [4] => writeimage
     [5] => getimageblob
     [6] => getimagegeometry
     [7] => getimageformat
     [8] => setimageformat
     [9] => setimagecompression
     [10] => setimagecompressionquality
     [11] => setimagepage
     [12] => scaleimage
     [13] => cropimage
     [14] => rotateimage
     [15] => flipimage
     [16] => flopimage
 )

 As I saw activity in here: https://core.trac.wordpress.org/ticket/21668
 only a couple hours ago by an admin after years of inactivity but dont
 know how to get into the slack conversation, I opened a ticket here. Also
 it would be nice if you would finally support progressive jpegs as stated
 in that ticket.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/35973>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform