Bug 96895

Summary: using tags images are sorted by name in slideshows even if order by date is selected
Product: [Applications] digikam Reporter: Sebastian Bothe <sbothe>
Component: Plugin-Generic-SlideShowAssignee: Digikam Developers <digikam-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: caulier.gilles
Priority: NOR    
Version First Reported In: 0.7.1   
Target Milestone: ---   
Platform: unspecified   
OS: Linux   
Latest Commit: Version Fixed/Implemented In: 7.6.0
Sentry Crash Report:

Description Sebastian Bothe 2005-01-13 10:45:40 UTC 
Version:           0.7.1 (using KDE 3.3.1,  (3.1))
Compiler:          gcc version 3.3.5 (Debian 1:3.3.5-3)
OS:                Linux (i686) release 2.6.9-1-686

While starting a slideshow using images in a tag the images are sorted by name always. This affects only the slideshow the preview is sorted correctly using the setting in 'view->sort images'
Comment 1 Renchi Raju 2005-01-13 21:30:39 UTC 
CVS commit by pahlibar: 


pick image list sorted as in current view for tag albums, when the list
is sent to kipi-plugins
BUG: 96895


  M +24 -14    kipiinterface.cpp   1.17


--- kdeextragear-3/digikam/digikam/kipiinterface.cpp  #1.16:1.17
@@ -366,4 +366,19 @@ KURL::List DigikamImageCollection::image
 KURL::List DigikamImageCollection::imagesFromTAlbum(TAlbum* album) const
 {
+    // if this album is the current album, then its already open
+    // just return whatevers visible in the albumitemhandler
+    if ( album == AlbumManager::instance()->currentAlbum() )
+    {
+        QStringList items;
+        AlbumItemHandler* handler =
+            AlbumManager::instance()->getItemHandler();
+        if (handler)
+            items = handler->allItemsPath();
+        
+        return KURL::List(items);
+    }
+
+    // else get the images from the database and return the items found
+
     AlbumDB* db = AlbumManager::instance()->albumDB();
 
@@ -626,10 +641,5 @@ void DigikamKipiInterface::slotSelection
 void DigikamKipiInterface::slotCurrentAlbumChanged( Album *album )
 {
-    bool b = false;
-    
-    if ( album )
-       b = true;
-    
-    emit currentAlbumChanged( b );
+    emit currentAlbumChanged( album != 0 );
 }