Bug 499191 - Adding a rating for a picture changes pick from pending to rejected
Summary: Adding a rating for a picture changes pick from pending to rejected
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Tags-Pick (other bugs)
Version First Reported In: 8.5.0
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2025-01-27 12:46 UTC by TD
Modified: 2025-04-14 11:28 UTC (History)
2 users (show)

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


Attachments
pick label settings (8.89 KB, image/png)
2025-01-28 09:28 UTC, TD
Details
sidecar options (6.57 KB, image/png)
2025-01-28 09:28 UTC, TD
Details
clip showing the unintended pick change (1.93 MB, video/mp4)
2025-01-28 09:29 UTC, TD
Details
sample picture (2.97 MB, image/jpeg)
2025-01-28 09:29 UTC, TD
Details
XMP metadata tab (before) (15.27 KB, image/png)
2025-01-28 11:45 UTC, TD
Details
XMP metadata tab (after) (16.59 KB, image/png)
2025-01-28 11:45 UTC, TD
Details

Note You need to log in before you can comment on or make changes to this bug.
Description TD 2025-01-27 12:46:52 UTC
***
If you're not sure this is actually a bug, instead post about it at https://discuss.kde.org

If you're reporting a crash, attach a backtrace with debug symbols; see https://community.kde.org/Guidelines_and_HOWTOs/Debugging/How_to_create_useful_crash_reports

Please remove this comment after reading and before submitting - thanks!
***

SUMMARY
When importing photos I use the pick feature and set them all to "pending". Afterwards I check the photos, assign tags and ratings and set the pick tag to "accepted" for good photos afterwards. With the current version 8.5.0 the pick tag is changed automatically to "rejected" when assigning a rating for the 1st time.

STEPS TO REPRODUCE
1. set all photos to "pending" in import dialogue
2. import photos
3. set any rating 1-5 stars for one imported photo

OBSERVED RESULT
The pick tag for that photo changes to "rejected".

EXPECTED RESULT
The pick tag should be unaffected.

SOFTWARE/OS VERSIONS
Windows: 11 Pro
macOS: 
(available in the Info Center app, or by running `kinfo` in a terminal window)
Linux/KDE Plasma: 
KDE Plasma Version: 
KDE Frameworks Version: 
Qt Version: 

ADDITIONAL INFORMATION
I already checked key shortcuts to make sure there is no double binding, but there is not.
Comment 1 Maik Qualmann 2025-01-27 20:39:51 UTC
I can't reproduce the problem. Are you using a sidecars? Maybe upload a sample image so we can test it.

Maik
Comment 2 Maik Qualmann 2025-01-27 20:43:38 UTC
Also check the Advanced Metadata settings. There are now settings for the pick label. "xmp.digiKam.PickLabel" should be at the top of the list.

Maik
Comment 3 TD 2025-01-28 09:27:25 UTC
Thank you for your answer. I am not using sidecars and xmp.digiKam.PickLabel is on top of the list. I've attached a short clip showing the current behaviour and also the sample picture (copy of the picture before changing the rating).

FYI: This only happens the first time I set a rating. If I change pick to "pending" afterwards and set a new rating, only the rating is affected.
Comment 4 TD 2025-01-28 09:28:18 UTC
Created attachment 177750 [details]
pick label settings
Comment 5 TD 2025-01-28 09:28:46 UTC
Created attachment 177751 [details]
sidecar options
Comment 6 TD 2025-01-28 09:29:24 UTC
Created attachment 177752 [details]
clip showing the unintended pick change
Comment 7 TD 2025-01-28 09:29:40 UTC
Created attachment 177753 [details]
sample picture
Comment 8 Maik Qualmann 2025-01-28 11:34:27 UTC
I can't reproduce it with the sample image either. Can you please take a screenshot of the XMP Metadata Tab before applying a rating and a screenshot after?

Maik
Comment 9 TD 2025-01-28 11:45:07 UTC
Created attachment 177759 [details]
XMP metadata tab (before)
Comment 10 TD 2025-01-28 11:45:24 UTC
Created attachment 177760 [details]
XMP metadata tab (after)
Comment 11 TD 2025-01-28 11:56:08 UTC
As you can see "Pick Label" is 1. This would be "rejected" instead of 2 for "pending". But the "Labels" view lists those files as "pending" and the icon overlay is pending as well (as you can see in the short clip).

If I press Alt+2 (for "pending") on a photo twice, "Pick Label" changes to 2 and the meta data under "Dynamic Media" vanishes (Good = False, pick = -1). Maybe this is related to this element.

As a result the "Labels" view lists photos with "Pick Label" 1 and 2 as "pending" and the "rejected" stays empty.
Comment 12 Maik Qualmann 2025-01-28 12:02:21 UTC
OK, that's strange. Even before a rating is set, there is already a rejected pick label in the metadata. So after importing (there is still a pending pick label in the database) and applying the rating, the pick label must already have been changed in the metadata. At the moment I have no idea.

Maik
Comment 13 caulier.gilles 2025-04-11 18:13:50 UTC
Hi,

The 8.7.0 pre-release Windows installer from today have been rebuilt from
scratch with Qt 6.8.3, KDE 6.12, OpenCV 4.11 + CUDA support, Exiv2 0.28.5, ExifTool 13.27, ffmpeg 7, all image codecs updated to last version (jxl, avif, heif, aom, etc.).

Please try with this version to see if your problem still reproducible...

https://files.kde.org/digikam/

Thanks in advance
Best regards

Gilles Caulier
Comment 14 TD 2025-04-14 10:35:45 UTC
(In reply to caulier.gilles from comment #13)
> Hi,
> 
> The 8.7.0 pre-release Windows installer from today have been rebuilt from
> scratch with Qt 6.8.3, KDE 6.12, OpenCV 4.11 + CUDA support, Exiv2 0.28.5,
> ExifTool 13.27, ffmpeg 7, all image codecs updated to last version (jxl,
> avif, heif, aom, etc.).
> 
> Please try with this version to see if your problem still reproducible...
> 
> https://files.kde.org/digikam/
> 
> Thanks in advance
> Best regards
> 
> Gilles Caulier

With the new 8.7.0 installer everything is working as it should, except for previously imported images (with 8.5.0), which still carry "Pick Label" = 1, but show up under "Pending" in the "Labels" view lists (icon overlay is pending as well).

Thank you very much.
Comment 15 caulier.gilles 2025-04-14 11:28:26 UTC
Thanks for the feedback. Of course older images inherit of the previous properties assignement...