[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        |  Resolution:
 Keywords:                |     Focuses:
--------------------------+------------------------------
Description changed by SergeyBiryukov:

Old description:

> 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.

New description:

 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: #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#comment:1>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform