Bug 118055

Summary: combined view no longer sorted
Product: [Applications] akregator Reporter: Matt MacMahon <adastra>
Component: generalAssignee: kdepim bugs <pim-bugs-null>
Status: RESOLVED FIXED    
Severity: normal    
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: ---   
Platform: Fedora RPMs   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

Description Matt MacMahon 2005-12-10 08:54:54 UTC 
Version:            (using KDE KDE 3.5.0)
Installed from:    Fedora RPMs
OS:                Linux

In combined view, in KDE 3.4.* articles showed up in reverse chronological order.  Now they have no discernable order.  Changing the ordering in the Normal and Widescreen views does not affect the order of the articles per feed or per folder.
This is using RPMs from http://kde-redhat.sourceforge.net/ for Fedora Core 3.
Comment 1 Frank Osterfeld 2006-01-16 01:58:35 UTC 
SVN commit 498730 by osterfeld:

regression: don't mix up article order in combined view: sort the by date, as it was in 3.4.x
BUG: 118055


 M  +1 -0      ChangeLog  
 M  +1 -0      src/articleviewer.cpp  


--- branches/KDE/3.5/kdepim/akregator/ChangeLog #498729:498730
@@ -7,6 +7,7 @@
 
 Bug fixes:
 
+ 2006/01/16 Fix article order in Combined View: sort by date (reversed) (#118055) -fo
  2006/01/15 Fix parsing of Atom 1.0 feeds with escaped HTML in it: Don't show tags
             as text (#112491, #117938) -fo
  2006/01/10 Select next item in article list when deleting the selected article
--- branches/KDE/3.5/kdepim/akregator/src/articleviewer.cpp #498729:498730
@@ -637,6 +637,7 @@
         return slotClear();
 
     QValueList<Article> articles = m_node->articles();
+    qHeapSort(articles);
     QValueList<Article>::ConstIterator end = articles.end();
     QValueList<Article>::ConstIterator it = articles.begin();