aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMark Sapiro <mark@msapiro.net>2010-07-27 07:41:37 -0700
committerMark Sapiro <mark@msapiro.net>2010-07-27 07:41:37 -0700
commit4bd962a8cab42a0262cceed71bcbd8b42294a293 (patch)
tree509417fd0a0ddcabeb82f9167d52f1444863b804
parentaeb387cbedd8c2a7791a42e0e387e2327b87e68b (diff)
downloadmailman2-4bd962a8cab42a0262cceed71bcbd8b42294a293.tar.gz
mailman2-4bd962a8cab42a0262cceed71bcbd8b42294a293.tar.xz
mailman2-4bd962a8cab42a0262cceed71bcbd8b42294a293.zip
Changed fix_url to lock the list if not locked. Bug #610364.
Diffstat (limited to '')
-rw-r--r--NEWS2
-rw-r--r--bin/fix_url.py7
2 files changed, 8 insertions, 1 deletions
diff --git a/NEWS b/NEWS
index 9fcaed8b..877c1c2b 100644
--- a/NEWS
+++ b/NEWS
@@ -42,6 +42,8 @@ Here is a history of user visible changes to Mailman.
Bug Fixes and other patches
+ - Changed fix_url to lock the list if not locked. Bug #610364.
+
- Made a minor change to the English subscribeack.txt (welcome message)
template to emphasize that a password is only required to unsubscribe
*without confirmation*.
diff --git a/bin/fix_url.py b/bin/fix_url.py
index 2d8dda58..d2731c14 100644
--- a/bin/fix_url.py
+++ b/bin/fix_url.py
@@ -1,6 +1,6 @@
#! @PYTHON@
#
-# Copyright (C) 2001,2002 by the Free Software Foundation, Inc.
+# Copyright (C) 2001-2010 by the Free Software Foundation, Inc.
#
# This program is free software; you can redistribute it and/or
# modify it under the terms of the GNU General Public License
@@ -69,6 +69,11 @@ def fix_url(mlist, *args):
elif opt in ('-v', '--verbose'):
verbose = 1
+ # Make sure list is locked.
+ if not mlist.Locked():
+ if verbose:
+ print _('Locking list')
+ mlist.Lock()
if urlhost:
web_page_url = mm_cfg.DEFAULT_URL_PATTERN % urlhost
mailhost = mm_cfg.VIRTUAL_HOSTS.get(urlhost.lower(), urlhost)