From aad73a5d90421d59ab71189004925cfc7fbf2884 Mon Sep 17 00:00:00 2001
From: Christopher Broderick <chris@chris-mbp>
Date: Fri, 10 Mar 2023 22:06:14 +0000
Subject: [PATCH] Fix format errors

---
 helpdesk/email.py | 67 +++++++++++++++++++++++++----------------------
 helpdesk/lib.py   |  4 +--
 2 files changed, 37 insertions(+), 34 deletions(-)

diff --git a/helpdesk/email.py b/helpdesk/email.py
index 428ffa70..af8ee822 100644
--- a/helpdesk/email.py
+++ b/helpdesk/email.py
@@ -499,41 +499,44 @@ def create_object_from_email_message(message, ticket_id, payload, files, logger)
         logger.info(
             "Message seems to be auto-reply, not sending any emails back to the sender")
     else:
-        # send mail to appropriate people now depending on what objects
-        # were created and who was CC'd
-        # Add auto-reply headers because it's an auto-reply and we must
-        extra_headers = {
-            'In-Reply-To': message_id,
-            "Auto-Submitted": "auto-replied",
-            "X-Auto-Response-Suppress": "All",
-            "Precedence": "auto_reply",
-        }
-        if new:
-            ticket.send(
-                {'submitter': ('newticket_submitter', context),
-                 'new_ticket_cc': ('newticket_cc', context),
-                 'ticket_cc': ('newticket_cc', context)},
-                fail_silently=True,
-                extra_headers=extra_headers,
-            )
-        else:
-            context.update(comment=f.comment)
-            ticket.send(
-                {'submitter': ('newticket_submitter', context),
-                 'assigned_to': ('updated_owner', context)},
-                fail_silently=True,
-                extra_headers=extra_headers,
-            )
-            if queue.enable_notifications_on_email_events:
-                ticket.send(
-                    {'ticket_cc': ('updated_cc', context)},
-                    fail_silently=True,
-                    extra_headers=extra_headers,
-                )
-
+        send_info_email(message_id, f, context, queue, new)
     return ticket
 
 
+def send_info_email(message_id: str, f: FollowUp, ticket: Ticket, context: dict, queue: dict, new: bool):
+    # send mail to appropriate people now depending on what objects
+    # were created and who was CC'd
+    # Add auto-reply headers because it's an auto-reply and we must
+    extra_headers = {
+        'In-Reply-To': message_id,
+        "Auto-Submitted": "auto-replied",
+        "X-Auto-Response-Suppress": "All",
+        "Precedence": "auto_reply",
+    }
+    if new:
+        ticket.send(
+            {'submitter': ('newticket_submitter', context),
+             'new_ticket_cc': ('newticket_cc', context),
+             'ticket_cc': ('newticket_cc', context)},
+            fail_silently=True,
+            extra_headers=extra_headers,
+        )
+    else:
+        context.update(comment=f.comment)
+        ticket.send(
+            {'submitter': ('newticket_submitter', context),
+             'assigned_to': ('updated_owner', context)},
+            fail_silently=True,
+            extra_headers=extra_headers,
+        )
+        if queue.enable_notifications_on_email_events:
+            ticket.send(
+                {'ticket_cc': ('updated_cc', context)},
+                fail_silently=True,
+                extra_headers=extra_headers,
+            )
+
+
 def get_ticket_id_from_subject_slug(
     queue_slug: str,
     subject: str,
diff --git a/helpdesk/lib.py b/helpdesk/lib.py
index f08aa23d..d265963b 100644
--- a/helpdesk/lib.py
+++ b/helpdesk/lib.py
@@ -81,12 +81,12 @@ def text_is_spam(text, request):
     # This will return 'True' is the given text is deemed to be spam, or
     # False if it is not spam. If it cannot be checked for some reason, we
     # assume it isn't spam.
-    from django.contrib.sites.models import Site
-    from django.core.exceptions import ImproperlyConfigured
     try:
         from akismet import Akismet
     except ImportError:
         return False
+    from django.contrib.sites.models import Site
+    from django.core.exceptions import ImproperlyConfigured
     try:
         site = Site.objects.get_current()
     except ImproperlyConfigured: