Bug 335718 - MANAGER : removing a tag turns entries in "Quick Access List" to "All tags"
Summary: MANAGER : removing a tag turns entries in "Quick Access List" to "All tags"
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Tags-Manager (show other bugs)
Version: 4.1.0
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-06-02 20:36 UTC by Michael G. Hansen
Modified: 2017-07-29 07:53 UTC (History)
2 users (show)

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


Attachments
Duplicate entries after deletion (39.41 KB, image/png)
2014-06-08 13:54 UTC, Michael G. Hansen
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael G. Hansen 2014-06-02 20:36:47 UTC
Reproducible: Always

Steps to Reproduce:
1. Create a tag "testtag" in the tree structure.
2. Add the tag "testtag" to the "Quick Access List".
3. Delete the tag "testtag" in the tree structure.
Actual Results:  
The item that was previously the entry for "testtag" in the "Quick Access List" now reads "All tags". It can not be removed from the "Quick Access List" via the context menu.

Expected Results:  
The removed tag should be removed from the "Quick Access List".

Closing and re-opening the tag editor clears the list from the broken tag references.
Comment 1 Veaceslav Munteanu 2014-06-08 10:31:37 UTC
Confirmed.. investigating...
Comment 2 Veaceslav Munteanu 2014-06-08 12:04:55 UTC
Git commit a7d349d08601f48cacec2389047d3d7c40407b0f by Veaceslav Munteanu.
Committed on 08/06/2014 at 14:59.
Pushed by munteanu into branch 'master'.

M  +3    -1    NEWS
M  +0    -4    digikam/tags/tagsmanager/models/tagmngrlistmodel.cpp
M  +1    -0    digikam/tags/tagsmanager/taglist.cpp

http://commits.kde.org/digikam/a7d349d08601f48cacec2389047d3d7c40407b0f
Comment 3 Michael G. Hansen 2014-06-08 13:54:02 UTC
Created attachment 87069 [details]
Duplicate entries after deletion

Hi Veaceslav,

the entry in the quick access list is now properly deleted. However, if the tag corresponding to the currently active filter is deleted, some other filter, for example "All tags" should be selected IMHO. Currently, no new selection is made.

Also, if I add tags 1,2,3, then set a filter using quick access list on 2, then delete 2, the tree view shows 1,1,3 as shown in the screenshot...

Best regards,

Michael
Comment 4 Veaceslav Munteanu 2014-06-08 14:14:57 UTC
Hmm... actually, with previous commit I set to reset to default All Tags when a selected entry is deleted.. but the is not visual feedback that is selected... I will look into it..
Comment 5 Veaceslav Munteanu 2014-09-02 13:09:25 UTC
Git commit 57204f0cc849f4e4c6ccc5f2963a1f2f0a0ec4dd by Veaceslav Munteanu.
Committed on 02/09/2014 at 13:08.
Pushed by munteanu into branch 'master'.

M  +2    -1    NEWS
M  +4    -0    digikam/tags/tagsmanager/taglist.cpp

http://commits.kde.org/digikam/57204f0cc849f4e4c6ccc5f2963a1f2f0a0ec4dd