[spambayes-dev] FolderSelector problem in Define Filters dialog

[Kenny Pitt]

Recently I reinstalled Outlook using a different profile name.  Rather 
than copying my old plug-in .INI file to the new profile name, I decided 
to just redo my settings from scratch in the SpamBayes Manager.  When I 
went into Define Filters I found that the Browse buttons under Certain 
Spam and Possible Spam did *nothing*, while the Browse button for 
"Filter the following folders" worked fine.

I tracked the problem down to two fixes, for which I have attached 
diffs.  The first change to opt_processors.py allowed the FolderSelector 
dialog to display when Browse was clicked, but I couldn't click OK and 
the status did not update at the bottom of the dialog.  The second fix 
in FolderSelector.py seems to have corrected the problem, and I now have 
my spam folders selected and filtering enabled.

-- 
Kenny Pitt
-------------- next part --------------
Index: opt_processors.py
===================================================================
RCS file: /cvsroot/spambayes/spambayes/Outlook2000/dialogs/opt_processors.py,v
retrieving revision 1.2
diff -u -r1.2 opt_processors.py
--- opt_processors.py	10 Aug 2003 07:26:50 -0000	1.2
+++ opt_processors.py	11 Aug 2003 14:50:20 -0000
@@ -186,7 +186,7 @@
             if is_multi:
                 ids = self.option.get()
             else:
-                ids = [self.optin.get()]
+                ids = [self.option.get()]
             from dialogs import FolderSelector
             if self.option_include_sub:
                 cb_state = self.option_include_sub.get()
-------------- next part --------------
Index: FolderSelector.py
===================================================================
RCS file: /cvsroot/spambayes/spambayes/Outlook2000/dialogs/FolderSelector.py,v
retrieving revision 1.21
diff -u -r1.21 FolderSelector.py
--- FolderSelector.py	10 Aug 2003 07:26:49 -0000	1.21
+++ FolderSelector.py	11 Aug 2003 14:47:11 -0000
@@ -340,8 +340,8 @@
             # If single-select, the checked state is not used, just the
             # selected state.
             try:
-                h = win32gui.SendMessage(self.list, commctrl.TVM_GETSELECTEDITEM,
-                                         commctrl.TVGN_CARET, h)
+                h = win32gui.SendMessage(self.list, commctrl.TVM_GETNEXTITEM,
+                                         commctrl.TVGN_CARET, commctrl.TVI_ROOT)
             except win32gui.error:
                 return
             info = self._GetLVItem(h)