https://bugs.kde.org/show_bug.cgi?id=326538

caulier.gil...@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://commits.kde.org/dig
                   |                            |ikam/9b154ceeecb732ecb97d69
                   |                            |00ab0841e643639f4e
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED

--- Comment #9 from caulier.gil...@gmail.com ---
Git commit 9b154ceeecb732ecb97d6900ab0841e643639f4e by Gilles Caulier.
Committed on 15/08/2017 at 18:06.
Pushed by cgilles into branch 'master'.

patch from Yingjie Liu for fix face tag region if image is rotated.
The tag region in database is not expected when rotate the image. The reason :
function FaceGroup::aboutToSetInfo(const ImageInfo& info) in
https://cgit.kde.org/digikam.git/tree/utilities/facemanagement/facegroup.cpp#n359
invoked applyItemGeometryChanges() after transformation. Before
FaceGroup::aboutToSetInfo(const ImageInfo& info), the new coordination of tag
region has been written into database. However, in applyItemGeometryChanges(),
the tag region coordination of tag region will be change again in database
which caused a wrong coordination.
A new FaceGroup::aboutToSetInfoAfterRotate(const ImageInfo& info) is now used
instead of FaceGroup::aboutToSetInfo(const ImageInfo& info) which will not
invoke applyItemGeometryChanges(). Then the coordination of tag region in
database is true after image transformation.
Related: bug 376681, bug 381378

M  +4    -1    NEWS
M  +2    -1    app/views/imagepreviewview.cpp
M  +11   -0    utilities/facemanagement/facegroup.cpp
M  +2    -0    utilities/facemanagement/facegroup.h

https://commits.kde.org/digikam/9b154ceeecb732ecb97d6900ab0841e643639f4e

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to