[buddypress-trac] [BuddyPress Trac] #7019: EXIF Orientation Handling Extraneous

[buddypress-trac]

#7019: EXIF Orientation Handling Extraneous
------------------------------------------------+--------------------------
 Reporter:  andrewgtibbetts                     |       Owner:
     Type:  defect (bug)                        |      Status:  new
 Priority:  normal                              |   Milestone:  Awaiting
Component:  Component - Attachments             |  Review
 Severity:  normal                              |     Version:  2.5.0
 Keywords:  needs-unit-tests reporter-feedback  |  Resolution:
------------------------------------------------+--------------------------
Changes (by r-a-y):

 * keywords:  needs-unit-tests => needs-unit-tests reporter-feedback


Comment:

 > So, it seems the work done in 5089 now breaks image orientation.

 Can someone verify that WordPress 4.5 indeed breaks the image orientation
 support we added in #5089?  Our unit test created for orientation -
 `test_bp_attachment_get_image_data()` - does not fail.

 It looks like WP's changeset only works if Imagick is used.  Not sure if
 that's what BuddyPress uses as well.

 Also since BP 2.6 will require WP 4.1, perhaps we can remove our older WP
 3.9 backward compatibility checks for orientation.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/7019#comment:2>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac