Image orientation in plupload, using EXIF

[thecoalman]

Is it possible that the image published from the iPad displays wrong, and the same image published from another system displays correctly? It seems impossible.

EXIF orientation can be quite confusing if not handled properly. For example suppose you were using editing software that doesn't support it. The image would be oriented wrong. You rotate the raw file data and save, the EXIF orientation is preserved . Loks fine in the editor but when you view it in something that does support it now it's flipped again. LOL... That's worse case because the orientation is no longer accurate.

How this applies here is as I just mentioned in previous post the thumbnails are not handled by plupload. The EXIF data is stripped when thumbnails are created so they will always have wrong orientation if they are oriented with EXIF.

[KYPREO]

But, the attachments don't render in post with the correct orientation. However, if you open the direct URL of the uploaded image in a new tab it is correctly oriented. The means the EXIF data is just being retained and properly interpreted by the browser, but not phpBB.

Plupload only handles the full size image it doesn't handle the thumbnail.

Thanks for the reply. I don't think that's it because I don't use thumbnails for attachments on my forum at all. The file size attachment is not being rendered with correct rotation within phpBB.

[KYPREO]

Well I just managed to fix it!

I implemented this change to /phpbb/plupload/plupload.php (around line 269):

From

Code: Select all

'resize: {width: %d, height: %d, quality: 85},',

to

Code: Select all

'resize: {width: %d, height: %d, quality: 85,preserve_headers: false},',

Previously this edit did nothing for me. Now, with the new version of plupload it works. Image is correctly orientated according to EXIF data and then the EXIF data is then stripped.

I know v12mike was able to get it working without this code change, but for me it was required. I will do some more testing, but all the test images and sample photos taken in landscape mode on my Samsung S9+ now display as they should.

[benwtf]

Testing with an image provided by a user who complained, I was able to use this extension with max dimensions set in attachments with no code modifications in 3.2.4.

Thank you!

EDIT: The extension here: viewtopic.php?f=456&t=2477256&p=15143616#p15143616
That's where I meant to post

[Streamer76]

Phpbb 3.2.5 seems to have plupload 2.3.6 version about plupload.full.min.js but there's still the same problem wit the images.

I added to "preserve_headers: false" to /phpbb/plupload/plupload.php but does not help. Somehow this trick worked fine with 3.1.x but not for the 3.2.5, at least not my phone images (LG G6).

Cannot use ImageMagick extension because my host does not support it.

Any ideas?

[canonknipser]

Any ideas?

Do it the old fashioned way: rotate (and maybe resize also) the image with a image manipulation program (gimp, photoshop, paint ...) or use jpegtran or a web service before uploading the images. I recommend it this way because rotating with plupload will only happen when the images needs to be resized. And the resizing with plupload is using a poor quality algorithm.

[Streamer76]

Any ideas?

Do it the old fashioned way: rotate (and maybe resize also) the image with a image manipulation program (gimp, photoshop, paint ...) or use jpegtran or a web service before uploading the images. I recommend it this way because rotating with plupload will only happen when the images needs to be resized. And the resizing with plupload is using a poor quality algorithm.

I can do that but I know that 99% users does not want to do that.

[canonknipser]

Some small questions:

• Is image resizing in effect on your board, which means do you have set maximum image dimensions?
• Is image resizing in effect for the pictures you mentioned

  at least not my phone images (LG G6).

  - are they bigger in dimension than those values set in acp?
• Do you generate thumbnails?
• Why does you host not support ImageMagick?

[GentekPower]

I wanted to add some input about the same issues I was having and the fix that finally got them...

Same rotation issues for me. Most were 90 deg counterclockwise. I tried the new bit of code with no success and then also the new version of plupload with what seemed no success...

The trick was that I had to purge the cache on both the server, phhpbb, AND MY BROWSER... That's what really fixed it. I had not been clearing the cached files on my local browser, but continuing to test and it wasn't working. For now it seems to be working... Board is running 3.2.3

[Streamer76]

Some small questions:

• Is image resizing in effect on your board, which means do you have set maximum image dimensions?
• Is image resizing in effect for the pictures you mentioned

  at least not my phone images (LG G6).

  - are they bigger in dimension than those values set in acp?
• Do you generate thumbnails?
• Why does you host not support ImageMagick?

Here is our board settings. I have no idea why my host does not support ImageMagick.

[Streamer76]

This is really annoying problem. Any ideas on this? (cannot test this on this forum because of file size limit, I can send someone some pictures if someone is willing to test those on some other board)

[Streamer76]

Any new ideas to solve this

[KYPREO]

Any new ideas to solve this

The image rotation based on EXIF is performed by plupload. This only happens when you set maximum image dimensions which then forces plupload to reduce the image size.

In your settings, maximum image size is 0 x 0 so plupload is not engaged.

Change the maximum image dimensions in the ACP to say 960x720.

Now try uploading an image again.

In order to fix image rotation when you don't have a maximum image dimension set, you need to use this extension: viewtopic.php?f=456&t=2477256

[Streamer76]

Thanks!