summaryrefslogtreecommitdiff
path: root/src/mailman/model/member.py
diff options
context:
space:
mode:
authorAurélien Bompard2016-02-09 12:23:38 +0100
committerBarry Warsaw2016-03-31 18:29:27 -0400
commit71cfeda5fb51c49fd87b6d34e00221a3725b9ec0 (patch)
treed571db90e7010643ea7cca81ab91eaee89e0f378 /src/mailman/model/member.py
parent011d278abcc25d417673f70a4eac0a8a78137d95 (diff)
downloadmailman-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.py10
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(