diff options
| author | bwarsaw | 1999-01-09 05:57:17 +0000 |
|---|---|---|
| committer | bwarsaw | 1999-01-09 05:57:17 +0000 |
| commit | a057fdc8e32c39230e18e4396c0d5832fcd8689b (patch) | |
| tree | f55c76342ecdc260b5b49baea108ae462141d157 /Mailman/Cgi/admin.py | |
| parent | b493e3bf310113b3bd88fe622f515e3157dedf57 (diff) | |
| download | mailman-a057fdc8e32c39230e18e4396c0d5832fcd8689b.tar.gz mailman-a057fdc8e32c39230e18e4396c0d5832fcd8689b.tar.zst mailman-a057fdc8e32c39230e18e4396c0d5832fcd8689b.zip | |
Diffstat (limited to 'Mailman/Cgi/admin.py')
| -rw-r--r-- | Mailman/Cgi/admin.py | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/Mailman/Cgi/admin.py b/Mailman/Cgi/admin.py index 13a1b811c..438e361c1 100644 --- a/Mailman/Cgi/admin.py +++ b/Mailman/Cgi/admin.py @@ -779,15 +779,6 @@ def ChangeOptions(lst, category, cgi_info, document): if not lst.nondigestable: digest = 1 try: - # catches blank lines or whitespace-only lines in mass - # subscribe dialog - if not new_name: - # TBD: we raise the exception here instead of just - # appending to subscribe_errors and doing a continue, - # because as of Python 1.5.2, this is not supported syntax - # (a continue inside the try inside a loop). - new_name = '<blank line>' - raise Errors.MMBadEmailError Utils.ValidateEmail(new_name) lst.ApprovedAddMember( new_name, @@ -797,6 +788,8 @@ def ChangeOptions(lst, category, cgi_info, document): except Errors.MMAlreadyAMember: subscribe_errors.append((new_name, 'Already a member')) except Errors.MMBadEmailError: + if new_name == '': + new_name = '<blank line>' subscribe_errors.append( (new_name, "Bad/Invalid email address")) except Errors.MMHostileAddress: |
