MediaWiki uses XMP data in addition to IFD0:Orientation to determine orientation
Closed, ResolvedPublic

Description

Author: saibotrash

Description:
( Similar to Bug #31487 )

[[:commons:File:David_Andrew_Jenkins_and_XAPP.jpg]] had (according to ExifTool) XMP-tiff:Orientation set to 8. XMP is IPTC Core (Adobe XMP)

Bug: Apparently Mediawiki uses not only IFD0:Orientation (contrary to https://bugzilla.wikimedia.org/show_bug.cgi?id=31504#c8 )

Btw: The image viewer gThumb which also rotates images based on EXIF info didn't rotate the image (correct).

own testcase... http://commons.wikimedia.org/wiki/File:AXMPtiffbeach.jpg with no EXIF data but the XMP-tiff:Orientation tag copyied over from [[:commons:File:David_Andrew_Jenkins_and_XAPP.jpg]]
Expected: the image shows up in its physical orientation (that is 90° CW).
Observed: 180°CW as MW applies apparently rotation based on the Orientation tag.

Reason: Either Mediawiki uses not only IFD0:Orientation (contrary to https://bugzilla.wikimedia.org/show_bug.cgi?id=31504#c8 ) or Exiftool and https://addons.mozilla.org/de/firefox/addon/exif-viewer/ do something really wrong here since they say that the image has no EXIF.

Problem: As we now use exiftool to read the IFD0-Orientation tag to be able to rotate images according to user expectations this breaks Rotatebot in Commons.


Version: 1.18.x
Severity: normal

bzimport added a subscriber: wikibugs-l.
bzimport set Reference to bz31637.
bzimport created this task.Via LegacyOct 12 2011, 2:35 AM
Bawolff added a comment.Via ConduitOct 12 2011, 12:58 PM

MediaWiki will use Orientation information from XMP as well (Specifically, the orientation tag in the http://ns.adobe.com/tiff/1.0/ namespace. The http://ns.adobe.com/tiff/1.0/ namespace in XMP is for encoding exif information in XMP form). If there is a conflict between the XMP tiff:Orientation and the Exif Orientation tag in ifd0, the tag in IFD0 takes priority.

I'm unsure if the current behaviour is correct or should be changed.

bzimport added a comment.Via ConduitOct 12 2011, 1:22 PM

saibotrash wrote:

Thanks. But: Hmpf! .. that gets really complicated... That EXIF-based rotation deployment is throughout a failure. Not even the devs know what really happens - apparently. :-(

Should we adjust Rotatebot now to also take XMP into account or will this be removed in MediaWiki?

Bawolff added a comment.Via ConduitOct 12 2011, 1:45 PM

Should we adjust Rotatebot now to also take XMP into account or will this be
removed in MediaWiki?

Well in theory, if you're rotating something, you should adjust all the rotation tags for complete correctness. Adjusting the XMP will never hurt anything

As for if using XMP's tiff:Orientation should be removed from MediaWiki - Good question. I don't know what the correct thing to do here is?

bzimport added a comment.Via ConduitOct 12 2011, 2:12 PM

saibotrash wrote:

half off topic -
(In reply to comment #3)
Bawolf: "Well in theory, if you're rotating something, you should adjust all the rotation tags for complete correctness."
(as written in IRC) There is just a minor problem: How should I know and understand any metadata format which could be used? I guess there are many more. I could strip all except jpeg data. But this would make users very unhappy.

Bawolff added a comment.Via ConduitOct 12 2011, 2:14 PM

I've disabled extracting tiff:Orientation in r99632

Gilles added a project: Multimedia.Via WebDec 4 2014, 9:34 AM
Gilles triaged this task as "Unbreak Now!" priority.Via WebDec 4 2014, 10:10 AM
Gilles moved this task to Closed on the Multimedia workboard.
Gilles lowered the priority of this task from "Unbreak Now!" to "Needs Triage".Via ConduitDec 4 2014, 11:22 AM

Add Comment

Column Prototype
This is a very early prototype of a persistent column. It is not expected to work yet, and leaving it open will activate other new features which will break things. Press "\" (backslash) on your keyboard to close it now.