Bug 38647 - incorrect unread article count
Summary: incorrect unread article count
Status: RESOLVED UNMAINTAINED
Alias: None
Product: knode
Classification: Miscellaneous
Component: general (show other bugs)
Version: 0.6.1
Platform: openSUSE Linux
: NOR normal
Target Milestone: ---
Assignee: kdepim bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2002-02-25 05:03 UTC by Peter Dowsett
Modified: 2018-09-04 18:38 UTC (History)
6 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Peter Dowsett 2002-02-25 04:58:53 UTC
(*** This bug was imported into bugs.kde.org ***)

Package:           knode
Version:           0.6.1 (using KDE 2.2.2 )
Severity:          normal
Installed from:    SuSE
Compiler:          gcc version 2.95.2 19991024 (release)
OS:                Linux (i586) release 2.2.19
OS/Compiler notes: 

Cosmetic bug only.

Sometimes Knode reports a number of unread articles in a group - even though none are visible with the "threads with unread" filter is selected.

The problem goes away when the "all" filter is used and the ignored threads are unignored then reignored.

(Submitted via bugs.kde.org)
(Called from KBugReport dialog)
Comment 1 Cp Hennessy 2002-09-29 04:27:13 UTC
This problem is still present in the current CVS version of knode.       
The problem lies in  KNArticleWidget as it contains a timer (t_imer)      
which when it times out causes the decrementation of the no of     
acticels read counter.      
However if you select an article, and then before the timer time's out     
you select another article, the timer gets reset.     
The following patch works but may not be the correct solution.   
Basically if the timer is still active then stop it and call by hand the   
method which would be called if the timer had finished.  
  Index: knarticlewidget.cpp 
=================================================================== 
RCS file: /home/kde/kdenetwork/knode/knarticlewidget.cpp,v 
retrieving revision 1.139 
diff -u -3 -p -u -r1.139 knarticlewidget.cpp 
--- knarticlewidget.cpp 19 Sep 2002 20:53:37 -0000      1.139 
+++ knarticlewidget.cpp 29 Sep 2002 02:13:45 -0000 
@@ -875,6 +875,10 @@ void KNArticleWidget::updateContents() 
 
 void KNArticleWidget::setArticle(KNArticle *a) 
 { 
+  if ( t_imer -> isActive() ) { 
+     t_imer->stop(); 
+     slotTimeout(); 
+     } 
   if(a_rticle && a_rticle->isOrphant()) 
     delete a_rticle; //don't leak orphant articles 
 
@@ -883,7 +887,6 @@ void KNArticleWidget::setArticle(KNArtic 
   r_ot13=false; 
   a_ctToggleRot13->setChecked(false); 
 
-  t_imer->stop(); 
 
   if(!a) 
     showBlankPage(); 
 
Comment 2 Cp Hennessy 2002-09-29 04:34:45 UTC
*** Bug 31719 has been marked as a duplicate of this bug. ***
Comment 3 Cp Hennessy 2002-10-12 13:35:42 UTC
Fixed in CVS 
Comment 4 Christian Gebauer 2002-10-16 22:36:27 UTC
 sorry, this patch doesn't fix the problem 
Comment 5 Cp Hennessy 2002-10-29 01:30:52 UTC
Christian suggests that this is a "misfeature" :) 
I tend to agree - based on my own usage. 
Christian also suggests that this feature is removed in KDE 3.2 
which seems reasonable. 
Comment 6 Rainer Endres 2004-04-05 12:39:28 UTC
Was this done for 3.2?
Comment 7 Alex Alexandrov 2007-11-29 13:28:44 UTC
still actual for 0.10.5 (kde 3.5.8). at least debian/ubuntu are affected.
Comment 8 Andrew Crouthamel 2018-09-04 18:38:22 UTC
Hello! Sorry to be the bearer of bad news, but this project has been unmaintained for many years so I am closing this bug.