From befa544becb8c336d8e14fa58f49faaefb1188dc Mon Sep 17 00:00:00 2001
From: Sachi King <nakato@nakato.io>
Date: Sun, 5 Mar 2017 18:46:20 +1100
Subject: [PATCH] Use outbound mail provider for admin BCC

We can use our outbound provider to BCC any required administive
retention that is required.
---
 pinaxcon/monkey_patch.py | 88 ----------------------------------------
 1 file changed, 88 deletions(-)

diff --git a/pinaxcon/monkey_patch.py b/pinaxcon/monkey_patch.py
index a85fe1e6..8192348d 100644
--- a/pinaxcon/monkey_patch.py
+++ b/pinaxcon/monkey_patch.py
@@ -1,5 +1,4 @@
 from django.conf import settings
-from django.core.mail import EmailMultiAlternatives  # noqa: F401
 from functools import wraps
 
 
@@ -12,7 +11,6 @@ class MonkeyPatchMiddleware(object):
 
 def do_monkey_patch():
     patch_speaker_profile_form()
-    patch_mail_to_send_bcc()
     fix_sitetree_check_access_500s()
     never_cache_login_page()
     patch_stripe_payment_form()
@@ -34,92 +32,6 @@ def patch_speaker_profile_form():
     fields["accessibility"].widget = widgets.AceMarkdownEditor()
 
 
-def patch_mail_to_send_bcc():  # noqa: C901
-    ''' Patches django.core.mail's message classes to send a BCC e-mail to
-    the default BCC e-mail address. '''
-
-    from django.core.mail import message
-
-    ARG = "bcc"
-
-    if hasattr(settings, "ENVELOPE_BCC_LIST"):
-        bcc_email = settings.ENVELOPE_BCC_LIST
-    else:
-        return  # We don't need to do this patch.
-
-    def bcc_arg_position(f):
-        ''' Returns the position of 'bcc' in the argument list to f, or None if
-        there is no such argument. '''
-        co = f.__code__
-        # The first co_argcount members of co_varnames are argument variables
-        for i, argname in enumerate(co.co_varnames[:co.co_argcount]):
-            if argname == ARG:
-                return i
-        else:
-            return None
-
-    def bcc_is_provided_positionally(f, a):
-        ''' Returns true if 'bcc' is provided as a positional argument to f,
-        when it is called with the argument list `a`. '''
-
-        return bcc_arg_position(f) < len(a)
-
-    def patch_bcc_positional(f, a):
-        ''' Returns a copy of `a`, but with the bcc argument patched to include
-        our BCC list. '''
-
-        pos = bcc_arg_position(f)
-        bcc = a[pos]
-
-        if bcc is not None:
-            bcc = list(bcc)
-        else:
-            bcc = []
-
-        bcc += bcc_email
-
-        return tuple(a[:pos] + (bcc,) + a[pos + 1:])
-
-    def patch_bcc_keyword(f, k):
-        ''' Adds our BCC list to the BCC list in the keyword arguments, and
-        returns the new version of the keyword arguments.
-
-        Arguments:
-            f (callable): The function that we're patching. It should have an
-                argument called bcc.
-            k (dict): A dictionary of kwargs to be provided to EmailMessage.
-            It will be modified to add the BCC list specified in
-            settings.ENVELOPE_BCC_LIST, if provided.
-        '''
-
-        if ARG in k:
-            bcc = list(k[ARG])
-            del k[ARG]
-        else:
-            bcc = []
-        bcc += list(bcc_email)
-        k[ARG] = bcc
-
-        return k
-
-    to_wrap = message.EmailMessage.__init__
-
-    @wraps(to_wrap)
-    def email_message__init__(*a, **k):
-
-        if bcc_is_provided_positionally(to_wrap, a):
-            a = patch_bcc_positional(to_wrap, a)
-        else:
-            k = patch_bcc_keyword(to_wrap, k)
-
-        return to_wrap(*a, **k)
-
-    message.EmailMessage.__init__ = email_message__init__
-
-    # Do not need to wrap EmailMultiAlternatives because it is a subclass of
-    # EmailMessage.
-
-
 def fix_sitetree_check_access_500s():
     ''' django-sitetree has a bug: https://github.com/idlesign/django-sitetree/pull/167/files
     -- it swallows the cause of all 500 errors. This swallows KeyErrors from