Didn't fix every EXIF orientation

Looking at the code, this module currently only seems to rotate 90, 180 or 270 degrees. It relies on the $image->rotate($degrees) method to do this.

Images in the example described in the issue summary that are not "fixed" by this module are flipped (and sometimes also rotated). However, apparently Drupal core does not provide a flip method yet.

So before we can implement this feature, I think we need to have this core issue committed first: ✨ Provide flip() method for image handling Needs work

This might be a duplicate of 🐛 Flipped orientation flags ignored RTBC .

Maybe this exif handling could just be put in core as well? It is never correct behavior to ignore and strip these tags. Especially when this is common when uploading images from iphones.

I'm seeing this works when using GD image toolkit. However using imagemagick (via ImageMagick → ) doesn't really work. It seems that imagemagick does some level of orientation itself.

Related to this? https://www.drupal.org/project/exif_orientation/issues/2895434 🐛 The module doesn't work with Imagick toolkit Active

Seems like a duplicate of 🐛 Flipped orientation flags ignored RTBC , where there is an RTBC patch