[Mailman-Users] Re: [Mailman-Developers] MM 2.0.2: AttributeError: MMAlreadyMember

[Thomas Wouters]

On Mon, Mar 12, 2001 at 03:35:32PM +0100, Detlef Neubauer wrote:
> Thomas Wouters <thomas@xs4all.net> writes:
> 
> > On Wed, Mar 07, 2001 at 05:11:41PM -0500, Mentor Cana wrote:
> > 
> > > Here is the error I get when attempting to approve subscription Request on
> > > "confirm+approval" list.
> > 
> > > admin(15231):   File "/opt/home/mailman/Mailman/ListAdmin.py", line 331, in __handlesubscription
> > > admin(15231):     except Errors.MMAlreadyMember:
> > > admin(15231): AttributeError: MMAlreadyMember
> > > ....
> > 
> > My typo. You can fix it manually by changing line 331 of
> > ListAdmin.py to 'except Errors.MMAlreadyAMember:'. Note the extra
> > 'A' between 'Already' and 'Member'.

> Hello,

> i have the same problem and i fix it. But now i get

> ,----[ log/error ]
> | Mar 12 15:19:33 2001 admin(4109): @@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@@
> | admin(4109): [----- Mailman Version: 2.0.2 -----]
> | admin(4109): [----- Traceback ------]
> | admin(4109): Traceback (innermost last):
> | admin(4109):   File "/home/mailman/scripts/driver", line 96, in run_main
> | admin(4109):     main()
> | admin(4109):   File "/home/mailman/Mailman/Cgi/admindb.py", line 88, in main
> | admin(4109):     HandleRequests(mlist, doc, cgidata)
> | admin(4109):   File "/home/mailman/Mailman/Cgi/admindb.py", line 271, in HandleRequests
> | admin(4109):     preserve, forward, forwardaddr)
> | admin(4109):   File "../Mailman/ListAdmin.py", line 133, in HandleRequest
> | admin(4109):     status = self.__handlesubscription(data, value, comment)
> | admin(4109):   File "../Mailman/ListAdmin.py", line 330, in __handlesubscription
> | admin(4109):     self.ApprovedAddMember(addr, password, digest, lang)
> | admin(4109): NameError: lang

> when i will to approve a new member. The refuse funktion is working.

Indeed. Barry and I found the same bug when we were preparing Mailman 2.0.3
last Thursday. This is the last time I'm using CVS to backport a patch, by
the way. The solution is to remove the last argument in the call to
'self.ApprovedAddMember', turning

        self.ApprovedAddMember(addr, password, digest, lang)

into

        self.ApprovedAddMember(addr, password, digest)

Barry was supposed to release 2.0.3 ASAPly, but I guess he's been as
devastated by jetlag as me ;P

-- 
Thomas Wouters <thomas@xs4all.net>

Hi! I'm a .signature virus! copy me into your .signature file to help me spread!