Bug 117401

Summary: digikam's 'my albums' and 'my tags' does no longer remember it's state when restarted
Product: [Applications] digikam Reporter: Achim Bohnet <ach>
Component: Albums-TreeViewAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version: 0.8.0   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed In: 0.9.0

Description Achim Bohnet 2005-12-01 00:35:09 UTC
Version:           0.8.0 (using KDE 3.4.3, Kubuntu Package 4:3.4.3-0ubuntu1 )
Compiler:          Target: i486-linux-gnu
OS:                Linux (i686) release 2.6.12-10-686

Looks like sometime in the past digikam did no
longer save (or load) the state of 'my albums'. At least
it now always opens with the same state when
I start digikam.

ah, and 'my tags' does remember it's state too.
'my date' is okay.

Achim
Comment 1 Tom Albers 2005-12-29 20:29:33 UTC
I can not reproduce it here....
Comment 2 Achim Bohnet 2005-12-29 23:53:27 UTC
Hi Tom,

I did some more test and found that all closed Albums that have still an
open child, are on restart always open again.  Only when all subalbums are closed
too, the open/close state of an album is remembered on restart.

Achim
Comment 3 Joern Ahrens 2005-12-30 00:33:45 UTC
SVN commit 492444 by jahrens:

BUG: 117401

 M  +2 -2      folderview.cpp  


--- trunk/extragear/graphics/digikam/digikam/folderview.cpp #492443:492444
@@ -314,8 +314,8 @@
     }
     
     FolderItem *item;    
-    QListViewItemIterator it(this);
-    for( ; it.current(); ++it)
+    QListViewItemIterator it(this->lastItem());
+    for( ; it.current(); --it)
     {
         item = dynamic_cast<FolderItem*>(it.current());
         if(!item)
Comment 4 Joern Ahrens 2005-12-30 00:37:45 UTC
SVN commit 492446 by jahrens:

backported...
CCBUG: 117401


 M  +2 -2      folderview.cpp  


--- branches/stable/extragear/graphics/digikam/digikam/folderview.cpp #492445:492446
@@ -314,8 +314,8 @@
     }
     
     FolderItem *item;    
-    QListViewItemIterator it(this);
-    for( ; it.current(); ++it)
+    QListViewItemIterator it(this->lastItem());
+    for( ; it.current(); --it)
     {
         item = dynamic_cast<FolderItem*>(it.current());
         if(!item)