Changeset 54265

Timestamp:

09/20/2022 04:57:15 PM (4 years ago)

Author:

antpb

Message:

Media: Respect EXIF Rotations.

Previously, the logic to determine EXIF rotation information was not providing the correct rotation. This patch respects the information properly by swapping some of the flip() logic on certain rotations.

Props tbember, SergeyBiryukov, costdev, mikeschroder, adamsilverstein.
Fixes #54937.

File:

• trunk/src/wp-includes/class-wp-image-editor.php (modified) (4 diffs)

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

@@ -513 +513 @@
                         case 2:
                                 // Flip horizontally.
-                                $result = $this->flip( true, false );
+                                $result = $this->flip( false, true );
                                 break;
                         case 3:
@@ -522 +522 @@
                         case 4:
                                 // Flip vertically.
-                                $result = $this->flip( false, true );
+                                $result = $this->flip( true, false );
                                 break;
                         case 5:
@@ -529 +529 @@
 
                                 if ( ! is_wp_error( $result ) ) {
-                                        $result = $this->flip( false, true );
+                                        $result = $this->flip( true, false );
                                 }
 
@@ -542 +542 @@
 
                                 if ( ! is_wp_error( $result ) ) {
-                                        $result = $this->flip( true, false );
+                                        $result = $this->flip( false, true );
                                 }