Overall status

Created on 30 September 2024, about 2 months ago

Doesn't work with D10.3 - we need the fix from 🐛 Use constraint instead of function validator Active
Doesn't work with imagemagick - see discussion in 🐛 The selected image handling toolkit 'imagemagick' can not process operation 'rotate'. Active
Doesn't handle flipped images - see discussion in 🐛 Flipped orientation flags ignored RTBC
When we adjust the image in hook_file_presave() it's "too late". The preview is wrongly oriented on the widget, and probably some thumbnail problems #3129355: Autorotated media items thumbnails distorted → .

Here are some suggestions

We remove exif_orientation_file_presave() and instead always add an upload validator. This simplifies the code and solves the "too late" problems. Hence we remove _exif_orientation_rotate() and the code can be directly in the FileImageExifOrientationConstraintValidator.
We could copy 4 files from image_effects to give the auto_orient and mirror operations. This significantly simplifies the validation code, removes the (now incorrect) orientation from the EXIF and it can handle flipped images. However what would happen if both modules are installed? Can we make ours lower priority? Maybe we pick different operation names??

🌱 Plan

Status

Active

Version

1.0

Component

Code

Created by

🇬🇧United Kingdom adamps

Live updates comments and jobs are added and updated live.

Merge Requests

Production build 0.71.5 2024