[Mailman-Developers] It happened again!

Barry A. Warsaw barry@zope.com
Wed, 16 Jan 2002 17:00:21 -0500


>>>>> "DM" == Dan Mick <dmick@utopia.West.Sun.COM> writes:

    DM> Dammit, last night I upgraded to latest CVS, and all my
    DM> nomails got reset again!  Argh.

    DM> ...and it's this line, right, here, right Barry?

Indeed.

    DM> Perhaps this is right:

Try this one instead.  And thanks for tracking this down!

-Barry

-------------------- snip snip --------------------
Index: versions.py
===================================================================
RCS file: /cvsroot/mailman/mailman/Mailman/versions.py,v
retrieving revision 2.17
diff -u -r2.17 versions.py
--- versions.py	2002/01/06 04:28:02	2.17
+++ versions.py	2002/01/16 21:58:55
@@ -302,6 +302,7 @@
         'bounce_you_are_disabled_warnings_interval',
         mm_cfg.DEFAULT_BOUNCE_YOU_ARE_DISABLED_WARNINGS_INTERVAL)
     add_only_if_missing('ban_list', [])
+    add_only_if_missing('delivery_status', {})
 
 
 
@@ -335,7 +336,6 @@
     # 2.1alpha3 -> 2.1alpha4.  The DisableDelivery flag is now moved into
     # get/setDeilveryStatus().  This must be done after the addresses are
     # canonicalized.
-    l.delivery_status = {}
     for k, v in l.user_options.items():
         if l.getMemberOption(k, mm_cfg.DisableDelivery):
             # Convert this flag into a legacy disable