Added a migration which changes the active bit from False to True to all email addresses associated with active drafts, and sends out notification emails about this.

 - Legacy-Id: 9166

Author: levkowetz

ietf/person/migrations/0004_auto_20150308_0440.py

@@ -0,0 +1,101 @@
+# -*- coding: utf-8 -*-
+from __future__ import unicode_literals
+
+#from django.db import models, migrations # pyflakes:ignore
+from django.db import migrations 
+from django.conf import settings
+
+from ietf.utils.mail import send_mail_text
+
+# Tiny detail: making the text below look nice is of course dependent on
+# how long the email addresses and draft name which are inserted into the
+# text actually turn out to be.  Rather than bothering to re-flow the text,
+# it can be noted that the average email address length, calculated over the
+# addresses this will be run for, was 21, and the length of the string
+# '%(inacive_email)s' is 18, so just formatting naturally should come out
+# pretty nicely in most cases.
+
+email_template = """
+Hi,
+
+As part of a review of active and inactive email addresses in the datatracker,
+it has been found that your address <%(inactive_email)s>, which is mentioned
+in the active draft %(draft_name)s, was set to inactive.
+
+As a consequence of this, it would not be receiving notifications related to
+that draft.  The notifications would instead have gone to <%(primary_email)s>.
+
+This most likely was not what you intended when you used <%(inactive_email)s>
+in the draft, so that address has now been set to active.  However, if you
+have manually set the status for <%(inactive_email)s> to inactive, we
+apologize for interfering and now having to ask you to go and set it to
+inactive again, at https://datatracker.ietf.org/accounts/profile/ .
+
+Best regards,
+
+    Henrik, via the inactive email migration script
+
+"""
+
+def activate_draft_email(apps, schema_editor):
+    Document = apps.get_model("doc", "Document")
+    print("Setting email addresses to active ...")
+    count = 0
+    for doc in Document.objects.filter(type__slug='draft', states__slug='active'):
+        for email in doc.authors.all():
+            if email.active == False:
+                primary = email.person.email_set.filter(active=True).order_by('-time').first()
+                email.active = True
+                email.save()
+                count += 1
+                # If there isn't a primary address, ther's no active
+                # addresses, and it can't be other than right to change the
+                # draft email address to active.  Otherwise, notify the owner.
+                if primary and settings.SERVER_MODE == 'production':
+                    primary_email = primary.address
+                    inactive_email = email.address
+                    context = dict(
+                        primary_email=primary_email,
+                        inactive_email=inactive_email,
+                        draft_name=doc.name,
+                        )
+                    send_mail_text(
+                        request=None,
+                        to=[ primary_email, inactive_email ],
+                        frm="Henrik Levkowetz <henrik@levkowetz.com>",
+                        subject="Changed email settings for you in the datatracker",
+                        txt= email_template % context,
+                        extra={"Reply-To": "Secretariat <ietf-action@ietf.org>"},
+                    )
+    print("Set %s email addresses to active" % count)
+
+def deactivate_draft_email(apps, scema_editor):
+    """
+    The backwards migration doesn't touch the active field of any email addresses.
+    We don't have the information to exactly undo what the forward migration did,
+    and on 08 Mar 2015, there were 1931 inactive email addresses coupled to active
+    drafts, and 4237 active addresses coupled to active drafts.  The harm would
+    be substantial if those active addresses were set to inactive.
+    """
+#     Document = apps.get_model("doc", "Document")
+#     print(" Not setting email addresses to inactive")
+#     count = 0
+#     for doc in Document.objects.filter(type__slug='draft', states__slug='active'):
+#         for email in doc.authors.all():
+#             if email.active == True:
+#                 #print email.address
+#                 count += 1
+#     print("Left %s active email addresses untouched " % count)
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ('person', '0003_auto_20150304_0829'),
+        ('doc', '0002_auto_20141222_1749'),
+    ]
+
+    operations = [
+        migrations.RunPython(
+            activate_draft_email,
+            deactivate_draft_email),
+    ]