From 386f269967c58faf403137b7778fb635dbb344a7 Mon Sep 17 00:00:00 2001
From: Andreas Gungl <a.gungl@gmx.de>
Date: Wed, 3 Nov 2004 20:41:53 +0000
Subject: [PATCH] Fix for two minor bugs: - classification was calculated over
 checked and unchecked tools - filter creation depended on the inclusion of
 classification rules

svn path=/trunk/kdepim/; revision=360187
---
 antispamwizard.cpp | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/antispamwizard.cpp b/antispamwizard.cpp
index 76d327137..c5414e1af 100644
--- a/antispamwizard.cpp
+++ b/antispamwizard.cpp
@@ -352,14 +352,13 @@ void AntiSpamWizard::accept()
       classHamFilter->setConfigureShortcut( TRUE );
       classHamFilter->setConfigureToolbar( TRUE );
       filterList.append( classHamFilter );
-
-      /* Now that all the filters have been added to the list, tell
-       * the filter manager about it. That will emit filterListUpdate
-       * which will result in the filter list in kmmainwidget being 
-       * initialized. This should happend only once. */
-      KMKernel::self()->filterMgr()->appendFilters( filterList );
-      
     }
+
+    /* Now that all the filters have been added to the list, tell
+     * the filter manager about it. That will emit filterListUpdate
+     * which will result in the filter list in kmmainwidget being 
+     * initialized. This should happend only once. */
+    KMKernel::self()->filterMgr()->appendFilters( filterList );
   }
 
   QDialog::accept();
@@ -377,13 +376,14 @@ void AntiSpamWizard::checkProgramsSelections()
     if ( mProgramsPage->isProgramSelected( (*it).getVisibleName() ) )
     {
       status = true;
-      if ( (*it).isSpamTool() )
+      if ( (*it).isSpamTool() ) {
         mSpamToolsUsed = true;
+        if ( (*it).useBayesFilter() )
+          canClassify = true;
+      }
       if ( (*it).isVirusTool() )
         mVirusToolsUsed = true;
     }
-    if ( (*it).useBayesFilter() )
-      canClassify = true;
   }
 
   if ( mSpamRulesPage )