Bug 421945

Summary:	Existing face tags not rotated with image
Product:	[Applications] digikam	Reporter:	David Griffiths <david>
Component:	Database-Faces	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	metzpinguin
Priority:	NOR
Version First Reported In:	7.0.0
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:	https://invent.kde.org/graphics/digikam/commit/99008600b7b2ba5b28ae15bec638c1eb8fad7b6d	Version Fixed/Implemented In:	7.0.0
Sentry Crash Report:
Attachments:	Photo that has been rotated - face tags in wrong place

Description David Griffiths 2020-05-23 00:16:16 UTC

SUMMARY
I had an imported image that already had face tags (Picasa). The image was inverted so I rotated it. The face tag bounding boxes were then in the wrong position (they were where the face was originally).

STEPS TO REPRODUCE
1. Have image with face tages
2. Rotate the image
3. Observe position of face tags

OBSERVED RESULT
Face tags in wrong position

EXPECTED RESULT
Face tags to move with rotated inage

SOFTWARE/OS VERSIONS
Windows: Win 10 64bit
macOS: 
Linux/KDE Plasma: 
(available in About System)
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION

Comment 1 Maik Qualmann 2020-05-23 04:19:03 UTC

Please upload a sample image. If images already contain face tags, the writing of face tags must be activated in the metadata settings. Otherwise digiKam cannot rotate the face tags.

Maik

Comment 2 David Griffiths 2020-05-23 12:41:13 UTC

Created attachment 128716 [details]
Photo that has been rotated - face tags in wrong place

Comment 3 Maik Qualmann 2020-05-23 13:04:24 UTC

I need the image before you rotate it. Another thing is, if you imported the image in your collection with an older digiKam version, the error would be explainable. I checked my original Picasa test imagers here again, there is no problem with/after the rotation.

Maik

Comment 4 David Griffiths 2020-05-24 14:11:36 UTC

Ok, I can find a copy before rotation - can you remind me the names that were on the image please.
I had not imported with any previous DigiKam - I have only just started using the program this weekend.

Comment 5 Maik Qualmann 2020-05-25 04:28:45 UTC

Git commit 99008600b7b2ba5b28ae15bec638c1eb8fad7b6d by Maik Qualmann.
Committed on 25/05/2020 at 04:26.
Pushed by mqualmann into branch 'master'.

fix face rotation when only change the orientation flag
FIXED-IN: 7.0.0

M  +2    -2    NEWS
M  +5    -0    core/libs/fileactionmanager/databaseworkeriface.cpp
M  +15   -99   core/libs/fileactionmanager/fileworkeriface.cpp
M  +0    -3    core/libs/fileactionmanager/fileworkeriface.h
M  +85   -0    core/libs/fileactionmanager/metadatahub.cpp
M  +7    -0    core/libs/fileactionmanager/metadatahub.h

https://invent.kde.org/graphics/digikam/commit/99008600b7b2ba5b28ae15bec638c1eb8fad7b6d