Bug 245061

Summary: [patch] "Restore Tag Filters" is broken, works only for top level nodes
Product: [Applications] digikam Reporter: Bastiaan Hovestreydt <bko>
Component: Albums-FiltersAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: minor    
Priority: NOR    
Version: 1.4.0   
Target Milestone: ---   
Platform: Compiled Sources   
OS: Linux   
Latest Commit: Version Fixed In: 1.4.0
Attachments: Patch to repair "Restore Tag Filters" feature

Description Bastiaan Hovestreydt 2010-07-18 18:32:48 UTC
Created attachment 49293 [details]
Patch to repair "Restore Tag Filters" feature

Version:           1.4.0 (using Devel) 
OS:                Linux

There seems to be missing a line of code to recurse the children of the tag tree in AbstractCheckableAlbumTreeView::restoreCheckStateForHierarchy.
I've attached a patch to fix this.

Reproducible: Always

Steps to Reproduce:
Set "Restore Tag Filters" to "On" in tag filter sidebar
Check some tags in the tag filter, which are not on the top level
Close Digikam
Start Digikam
Observe, that checked tags were not restored
Comment 1 Johannes Wienke 2010-07-18 20:26:27 UTC
SVN commit 1151296 by jwienke:

Fix restoring the check-state of tree views also for children.

Thanks for the patch.

BUG: 245061

 M  +2 -1      NEWS  
 M  +1 -0      digikam/albumtreeview.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1151296