Bug 436979 - Empty regions
Summary: Empty regions
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Metadata-Xmp (show other bugs)
Version: 7.3.0
Platform: Other Other
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-05-12 14:40 UTC by herb
Modified: 2021-05-13 08:24 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 7.3.0
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description herb 2021-05-12 14:40:14 UTC
SUMMARY
Hello

this report is similar/identical to 435340.

In case of I assign a tag (keyword) to a file, that does not contain any XMP-metadata, also region-fields are created: but they are empty.

I used the right panel: Captions--> Tags to assign it.


STEPS TO REPRODUCE
1. 
2. 
3. 

OBSERVED RESULT


EXPECTED RESULT


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

ADDITIONAL INFORMATION
Comment 1 Maik Qualmann 2021-05-12 17:40:16 UTC
The problem is not reproducible, regions of the face are also written correctly in an image that previously did not contain XMP metadata when a tag is assigned.

Please provide a test image with the problem occurring and/or a DebugView log while the tag is being assigned.

Maik
Comment 2 herb 2021-05-12 20:04:21 UTC
Hello Maik,

thanks for your investigations.
I agree that all regions are written correctly.

I regret, that I was not detailed enough; but I thought it is clear because of my hint with error 435340.


The problem is the same as in 435340.
In case of I
- assign a tag (keyword) and
- no face regions exist for this image
- also empty regions are written.

I think empty regions should not be created..
Comment 3 Maik Qualmann 2021-05-12 20:13:24 UTC
However, if the tag is a face tag, a Microsoft Photo 1.2 scheme is written. We discussed this with another Windows user. However, this possibility is actually described in the Microsoft documentation for face tags. I'll look for the discussion and then close the bug.

Maik
Comment 4 Maik Qualmann 2021-05-12 20:16:54 UTC
Read this: Bug 432048

Maik
Comment 5 herb 2021-05-13 07:38:17 UTC
(In reply to Maik Qualmann from comment #3)
> However, if the tag is a face tag, a Microsoft Photo 1.2 scheme is written.
> We discussed this ... 

Yes, that could be ok.
But in my case, NO facetag exists for the image.

Best regards
herb
Comment 6 Maik Qualmann 2021-05-13 08:24:44 UTC
Git commit 6bdfb6a8717d85db6d8a4015f867c935ead8b795 by Maik Qualmann.
Committed on 13/05/2021 at 08:23.
Pushed by mqualmann into branch 'master'.

dont create empty metadata face regions if they not exist
FIXED-IN: 7.3.0

M  +24   -12   core/libs/metadataengine/dmetadata/dmetadata_faces.cpp

https://invent.kde.org/graphics/digikam/commit/6bdfb6a8717d85db6d8a4015f867c935ead8b795