[Mailman-Developers] Some bugs... (HTMLFormatter and Cgi/admin)

[John Viega]

On Sun, Oct 11, 1998 at 05:16:20PM +0200, Bavo De Ridder wrote:
> Hello,
 > 
> I recently got the latest cvs-release. After installing it I got the
> following errors:
> 
> 	1) In HTMLFormatter.py (line 160) a call to a none
> 	   existsing attribute of MailList
> 
> 	   correction: changed the if to 
> 
> 		if self.subscribe_policy < 2 :
> 
> 	   I don't know if the "< 2" is completely correct, but
> 	   the real correction probably goes in that direction.

I don't understand you here.  What's the exact error you're getting?



> 	2) In Cgi/admin.py on line 766 there is an error, that line says
> 
> 	   list.digest_members.remove(user)
> 
> 	   The correction is:
> 
> 	   lst.digest_members.remove(user)
> 
> I do wonder why nobody hasn't seen these errors before... The error in
> admin.py was also in 1.04b so at least some of you should have noticed
> it... Doesn't anyone use the admin pages to set subscriber options?

Well, I just updated HTMLFormatter a little while ago, and haven't
tried it out yet.  I don't understand you problem with it yet, though.
As for the list->lst problem, I did notice that last week, but I
hadn't gotten around to making the change in my checked out version
and checking it in, I'd only made the fix in a running version.  Note
that the CVS tree is not garunteed to be in a state that will run
whatsoever, though we do try to keep it that way.  We only do lots of
testing when it comes time for an actual release. (And even then we
don't always catch everything).

John