[ mailman-Bugs-717096 ] Changing address "globally" if not on all lists fails

[SourceForge.net]

Bugs item #717096, was opened at 2003-04-07 17:09
Message generated for change (Tracker Item Submitted) made by Item Submitter
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=717096&group_id=103

Category: (un)subscribing
Group: 2.1 (stable)
Status: Open
Resolution: None
Priority: 5
Submitted By: Joe Peterson (skyrush)
Assigned to: Nobody/Anonymous (nobody)
Summary: Changing address "globally" if not on all lists fails

Initial Comment:
If you have a user who is on some, but not all, of the
lists on your server, and he/she tries to change
his/her address *globally* (using the "change globally"
checkbox), the person will get the confirmation email,
but when he/she clicks the link in the email and then
the button on the page to make the change, it will only
change the address on the list originally chosen, and
then it will raise a "Bad Confirmation code" error.

The problem is that the call,
"self.__assertIsMember(member)" in
OldStyleMemberships.py fails and raises the error.

The isMember() call should have been checked in
ApprovedChangeMemberAddress in MailList.py before
trying to change the address in the other lists.  This
statement fixes it:

            if not mlist.isMember(oldaddr):
                continue

Note that this sort of check *is* done in
ChangeMemberName (which I used as an example for the fix):

            if not mlist.isMember(addr):
                continue 

                                            -Joe


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=100103&aid=717096&group_id=103