Bug 149645 - insert photo album does not rotate images
Summary: insert photo album does not rotate images
Status: RESOLVED INSUFFICIENTDATA
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: Impress (show other bugs)
Version:
(earliest affected)
7.3.3.2 release
Hardware: All All
: medium enhancement
Assignee: Not Assigned
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2022-06-20 19:25 UTC by Simo He
Modified: 2023-01-19 03:25 UTC (History)
1 user (show)

See Also:
Crash report or crash signature:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Simo He 2022-06-20 19:25:26 UTC
Description:
When inserting images with the album functionality, images are not rotated according to the exif data.
Since bug 146019 is resolved, inserted images are rotated automatically matching to the exif data (without user confirmation) in almost all places. So I expected it here too.

Steps to Reproduce:
1. in menu, select Insert->Media->Picture Galery
2. select "insert" button in the dialog
3. select an image with exif rotation (eg. attachment 176569 [details])
3. click "open" (in the selection dialog)
4. click "insert slides" (in the album dialog)

Actual Results:
the image is inserted rotated (down is not down)

Expected Results:
the picture is inserted looking normal (so rotated according to the exif data)


Reproducible: Always


User Profile Reset: No



Additional Info:
I tested with 7.3.3.2 (and 7.0.4.2)

The automatic rotation without confirmation should be in 7.3.x according to https://bugs.documentfoundation.org/show_bug.cgi?id=146019#c3

In the code for photo album [1] some code like the following is probably missing:

    const Degree10 aRotation = aMetadata.getRotation();
    if (aRotation)
    {
        GraphicNativeTransform aTransform( aGraphic );
        aTransform.rotate( aRotation );
    }

code found in https://git.libreoffice.org/core/+/30f6ad6a617e7b9aa86f774e8b6da9c4ae8ad9f4%5E%21

[1] https://git.libreoffice.org/core/+/refs/heads/master/sd/source/ui/dlg/PhotoAlbumDialog.cxx#124
Comment 1 Rafael Lima 2022-06-20 22:35:58 UTC
Not repro in

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: cb83063cc0eb4e93bd44bc0cb9b7c4841230cdef
CPU threads: 16; OS: Linux 5.13; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Calc: threaded

Inserting the image will rotate it 90º CCW as informed in the exif data (the house looks correctly aligned).

Also not repro in

Version: 7.3.3.2 / LibreOffice Community
Build ID: 30(Build:2)
CPU threads: 16; OS: Linux 5.13; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Ubuntu package version: 1:7.3.3~rc2-0ubuntu0.21.10.1~lo1
Calc: threaded
Comment 2 Rafael Lima 2022-06-21 17:52:19 UTC
Simon, can you please try again on a clean profile? Or maybe on a different computer?
Comment 3 QA Administrators 2022-12-19 03:18:37 UTC Comment hidden (obsolete)
Comment 4 QA Administrators 2023-01-19 03:25:07 UTC
Dear Simo He,

Please read this message in its entirety before proceeding.

Your bug report is being closed as INSUFFICIENTDATA due to inactivity and
a lack of information which is needed in order to accurately
reproduce and confirm the problem. We encourage you to retest
your bug against the latest release. If the issue is still
present in the latest stable release, we need the following
information (please ignore any that you've already provided):

a) Provide details of your system including your operating
   system and the latest version of LibreOffice that you have
   confirmed the bug to be present

b) Provide easy to reproduce steps – the simpler the better

c) Provide any test case(s) which will help us confirm the problem

d) Provide screenshots of the problem if you think it might help

e) Read all comments and provide any requested information

Once all of this is done, please set the bug back to UNCONFIRMED
and we will attempt to reproduce the issue. Please do not:

a) respond via email 

b) update the version field in the bug or any of the other details
   on the top section of our bug tracker

Warm Regards,
QA Team

MassPing-NeedInfo-FollowUp