Changeset 40129 for trunk/src/wp-includes/class-wp-image-editor-imagick.php

Timestamp:

02/27/2017 04:21:41 AM (8 years ago)

Author:

mikeschroder

Message:

Media: After [40123], Feature check setImageOrientation.

In [40123], WP_Image_Editor_Imagick started using
Imagick::setImageOrientation and Imagick::ORIENTATION_TOPLEFT,
but had no equivalent feature check.

While they were introduced more than 9 years ago, it's important
to double-check everything is available before using with Imagick.

Fixes #37140.

File:

• trunk/src/wp-includes/class-wp-image-editor-imagick.php (modified) (1 diff)

trunk/src/wp-includes/class-wp-image-editor-imagick.php

@@ -551 +551 @@
 
             // Normalise Exif orientation data so that display is consistent across devices.
-            $this->image->setImageOrientation( Imagick::ORIENTATION_TOPLEFT );
+            if ( is_callable( array( $this->image, 'setImageOrientation' ) ) && defined( 'Imagick::ORIENTATION_TOPLEFT' ) ) {
+                $this->image->setImageOrientation( Imagick::ORIENTATION_TOPLEFT );
+            }
 
             // Since this changes the dimensions of the image, update the size.