------------------------------------------------------------
revno: 1360
committer: Mark Sapiro <[email protected]>
branch nick: 2.1
timestamp: Wed 2012-08-22 20:58:18 -0700
message:
The query fragments send_unsub_notifications_to_list_owner and
send_unsub_ack_to_this_batch will now assume default values if not set
in mass unsubscribe URLs. (LP: #1032378)
modified:
Mailman/Cgi/admin.py
NEWS*
--
lp:mailman/2.1
https://code.launchpad.net/~mailman-coders/mailman/2.1
Your team Mailman Checkins is subscribed to branch lp:mailman/2.1.
To unsubscribe from this branch go to
https://code.launchpad.net/~mailman-coders/mailman/2.1/+edit-subscription
=== modified file 'Mailman/Cgi/admin.py'
--- Mailman/Cgi/admin.py 2012-02-23 16:22:11 +0000
+++ Mailman/Cgi/admin.py 2012-08-23 03:58:18 +0000
@@ -1436,10 +1436,12 @@
removals += cgidata['unsubscribees_upload'].value
if removals:
names = filter(None, [n.strip() for n in removals.splitlines()])
- send_unsub_notifications = int(
- cgidata['send_unsub_notifications_to_list_owner'].value)
- userack = int(
- cgidata['send_unsub_ack_to_this_batch'].value)
+ send_unsub_notifications = safeint(
+ 'send_unsub_notifications_to_list_owner',
+ mlist.admin_notify_mchanges)
+ userack = safeint(
+ 'send_unsub_ack_to_this_batch',
+ mlist.send_goodbye_msg)
unsubscribe_errors = []
unsubscribe_success = []
for addr in names:
@@ -1463,11 +1465,7 @@
doc.AddItem('<p>')
# See if this was a moderation bit operation
if cgidata.has_key('allmodbit_btn'):
- val = cgidata.getvalue('allmodbit_val')
- try:
- val = int(val)
- except VallueError:
- val = None
+ val = safeint('allmodbit_val')
if val not in (0, 1):
doc.addError(_('Bad moderation flag value'))
else:
=== modified file 'NEWS' (properties changed: +x to -x)
--- NEWS 2012-06-27 00:53:52 +0000
+++ NEWS 2012-08-23 03:58:18 +0000
@@ -5,10 +5,14 @@
Here is a history of user visible changes to Mailman.
-2.1.15-1 (xx-xxx-xxxx)
+2.1.16 (xx-xxx-xxxx)
Bug Fixes and other patches
+ - The query fragments send_unsub_notifications_to_list_owner and
+ send_unsub_ack_to_this_batch will now assume default values if not set
+ in mass unsubscribe URLs. (LP: #1032378)
+
- Replaced utf-8 encoded characters in newly added German templates with
HTML entities. (LP: #1018208)
_______________________________________________
Mailman-checkins mailing list
[email protected]
Unsubscribe:
http://mail.python.org/mailman/options/mailman-checkins/archive%40jab.org
