Bug 118055 - combined view no longer sorted
Summary: combined view no longer sorted
Status: RESOLVED FIXED
Alias: None
Product: akregator
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Fedora RPMs Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-12-10 08:54 UTC by Matt MacMahon
Modified: 2006-01-16 01:58 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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();