diff options
| author | Aurélien Bompard | 2016-02-09 12:23:38 +0100 |
|---|---|---|
| committer | Barry Warsaw | 2016-03-31 18:29:27 -0400 |
| commit | 71cfeda5fb51c49fd87b6d34e00221a3725b9ec0 (patch) | |
| tree | d571db90e7010643ea7cca81ab91eaee89e0f378 /src/mailman/model/member.py | |
| parent | 011d278abcc25d417673f70a4eac0a8a78137d95 (diff) | |
| download | mailman-71cfeda5fb51c49fd87b6d34e00221a3725b9ec0.tar.gz mailman-71cfeda5fb51c49fd87b6d34e00221a3725b9ec0.tar.zst mailman-71cfeda5fb51c49fd87b6d34e00221a3725b9ec0.zip | |
Diffstat (limited to 'src/mailman/model/member.py')
| -rw-r--r-- | src/mailman/model/member.py | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/src/mailman/model/member.py b/src/mailman/model/member.py index 99f2de9b8..a6555c0fe 100644 --- a/src/mailman/model/member.py +++ b/src/mailman/model/member.py @@ -76,14 +76,10 @@ class Member(Model): raise ValueError('subscriber must be a user or address') if role in (MemberRole.owner, MemberRole.moderator): self.moderation_action = Action.accept - elif role is MemberRole.member: - self.moderation_action = getUtility(IListManager).get_by_list_id( - list_id).default_member_action else: - assert role is MemberRole.nonmember, ( - 'Invalid MemberRole: {}'.format(role)) - self.moderation_action = getUtility(IListManager).get_by_list_id( - list_id).default_nonmember_action + assert role in (MemberRole.member, MemberRole.nonmember), ( + 'Invalid MemberRole: {0}'.format(role)) + self.moderation_action = None def __repr__(self): return '<Member: {} on {} as {}>'.format( |
