[dreamwidth/dw-free] a227e3: Add automatic orientation of images

[github]

Branch: refs/heads/develop
Home: https://github.com/dreamwidth/dw-free
Commit: a227e3c26d34af66199322668dd65e77432c2013
https://github.com/dreamwidth/dw-free/commit/a227e3c26d34af66199322668dd65e77432c2013
Author: Mark Smith <mark@dreamwidth.org>
Date: 2013-08-12 (Mon, 12 Aug 2013)

Changed paths:
M bin/checkconfig.pl
M cgi-bin/DW/Media.pm
M cgi-bin/DW/Media/Photo.pm

Log Message:
-----------
Add automatic orientation of images

Depending on which way you hold your camera, the JPG image that comes
out might be in any number of 8 different orientations. Some browsers
handle these one way, some another, and at the end of the day it's all a
big mess.

Instead of relying on browsers to handle and display the images, we've
decided to orient them on upload. This will give images the best chance
of being viewed correctly wherever they are shown.

Comments

[fu]

Mwahaha.

[ninetydegrees]

This all so very exciting!

Error?

[ninetydegrees]

I think this causes an error message on hacks:

Error loading event module 'LJ::Event::JournalNewComment': Can't locate Image/ExifTool.pm in @INC

Edit: I guess because they don't have what this needs yet?

Re: Error?

[mark]

Yeah. There's a new module required. I added it to bin/checkconfig.pl but someone then has to make it available on the hacks. I don't know Sophie's preferred method of doing that.

In production, and on my own development environments, all I do is:

cpanm -L$LJHOME/extlib/ Image::ExifTool

That chugs for a while, then it's done and the module is installed locally.