[buddypress-trac] [BuddyPress Trac] #5089: Avatar rotation fix for iOS

[buddypress-trac]

#5089: Avatar rotation fix for iOS
-------------------------------------+------------------
 Reporter:  modemlooper              |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  2.4
Component:  Component - Attachments  |     Version:  1.0
 Severity:  normal                   |  Resolution:
 Keywords:  needs-testing has-patch  |
-------------------------------------+------------------

Comment (by imath):

 Thanks a lot @DJPaul for your feedback.

 About `exif_read_data` i've simply copy/pasted what WordPress was doing in
 `wp_read_image_metadata()` but you're right, i think we can remove the
 suppress errors thing.
 So .05 is taking in account your feedback.

 I've timed before/and after the 3 new unittests i'm suggesting and it
 seems the impact is not very significant:
 - not multisite: before = 1.17 minutes / after = 1.17 minutes
 - multisite: before = 1.59 minutes / after = 1.59 minutes

 Maybe we could try to include them and if we see it has a significant
 impact remove them ? What do you think ?

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