fix(SizeMetadataProvider): Swap the width and height if the image is …

…rotated

Signed-off-by: provokateurin <kate@provokateurin.de>

lib/AppInfo/Application.php

@@ -92,6 +92,7 @@ public function register(IRegistrationContext $context): void {
 		// Metadata
 		$context->registerEventListener(MetadataLiveEvent::class, ExifMetadataProvider::class);
 		$context->registerEventListener(MetadataBackgroundEvent::class, ExifMetadataProvider::class);
+		// SizeMetadataProvider optionally depends on ExifMetadataProvider, so it has to be registered afterwards
 		$context->registerEventListener(MetadataLiveEvent::class, SizeMetadataProvider::class);
 		$context->registerEventListener(MetadataBackgroundEvent::class, SizeMetadataProvider::class);
 		$context->registerEventListener(MetadataLiveEvent::class, OriginalDateTimeMetadataProvider::class);

lib/Listener/SizeMetadataProvider.php

@@ -67,6 +67,22 @@ public function handle(Event $event): void {
 			return;
 		}
 
+		// The image might have a rotation stored in the EXIF data.
+		// If that is the case and the rotation is 90/270 degrees the width and height need to be swapped.
+		// This is necessary because the clients will take the rotation into account when displaying the image.
+		if ($event->getMetadata()->hasKey('photos-ifd0')) {
+			$ifd0 = $event->getMetadata()->getArray('photos-ifd0');
+			if (array_key_exists('Orientation', $ifd0)) {
+				/** @var int $orientation */
+				$orientation = $ifd0['Orientation'];
+
+				// https://exiftool.org/TagNames/EXIF.html
+				if ($orientation >= 5) {
+					$size = [$size[1], $size[0]];
+				}
+			}
+		}
+
 		$event->getMetadata()->setArray('photos-size', [
 			'width' => $size[0],
 			'height' => $size[1],