Bug 382677 - Rating groups by mouse only changes first item [patch]
Summary: Rating groups by mouse only changes first item [patch]
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Tags-Rating (show other bugs)
Version: 5.6.0
Platform: Appimage Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-07-24 19:12 UTC by Alexander
Modified: 2017-08-06 23:33 UTC (History)
3 users (show)

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


Attachments
Considers grouping for rating applied via overlay (12.98 KB, patch)
2017-07-24 23:55 UTC, Simon
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander 2017-07-24 19:12:12 UTC
If you group items and you have metadata group edit enabled rating a group only works if you use the shortcuts (ctrl 0-5). If you use the mouse to change the stars of a group only the first item in the group gets rated.

This is serious imo because you will get metadata inconsistency if you use your mouse to change a rating.
Comment 1 Simon 2017-07-24 23:55:03 UTC
Created attachment 106845 [details]
Considers grouping for rating applied via overlay
Comment 2 Simon 2017-07-28 12:28:54 UTC
@Gilles, Maik, ...: Good to go? It fixed the bug for me.

Just to try out how it works I also posted this on phabricator:
https://phabricator.kde.org/D6962
You can either ignore it (should be closed automatically once committed) or use it to comment on the code.
Comment 3 Maik Qualmann 2017-07-29 15:02:18 UTC
Patch work fine for me, commit it.

Maik
Comment 4 Simon 2017-07-29 15:08:07 UTC
Git commit 0c7e25df54de368a8cd5f309380c6bbb1d659d51 by Simon Frei.
Committed on 29/07/2017 at 15:07.
Pushed by sfrei into branch 'master'.

grouping: Fix setting rating via overlay

Requires new functionality to check for grouping for specific model indexes
instead of currently selected ones. To that end some refactoring was necessary,
but existing function signatures have not changed.
FIXED-IN: 5.7.0

M  +7    -16   app/items/digikamimageview.cpp
M  +0    -2    app/items/digikamimageview.h
M  +69   -49   app/items/imagecategorizedview.cpp
M  +11   -1    app/items/imagecategorizedview.h

https://commits.kde.org/digikam/0c7e25df54de368a8cd5f309380c6bbb1d659d51
Comment 5 Ahmed Fathi 2017-08-06 23:33:12 UTC
Git commit 615b18b591ce199b392cbc0b0f2cbe90d47b09a2 by Ahmed Fathi Shaban, on behalf of Simon Frei.
Committed on 06/08/2017 at 23:14.
Pushed by ahmedfathy into branch 'gsoc17-dlna'.

grouping: Fix setting rating via overlay

Requires new functionality to check for grouping for specific model indexes
instead of currently selected ones. To that end some refactoring was necessary,
but existing function signatures have not changed.
FIXED-IN: 5.7.0

M  +7    -16   app/items/digikamimageview.cpp
M  +0    -2    app/items/digikamimageview.h
M  +69   -49   app/items/imagecategorizedview.cpp
M  +11   -1    app/items/imagecategorizedview.h

https://commits.kde.org/digikam/615b18b591ce199b392cbc0b0f2cbe90d47b09a2