1 files changed, 22 insertions, 17 deletions

diff --git a/core/middleware.py b/core/middleware.py
index c535291..54cdb11 100644
--- a/core/middleware.py
+++ b/core/middleware.py
@@ -5,9 +5,11 @@ from django.urls import Resolver404, resolve, reverse
 """Non-standard django middleware is loaded from this file.
 
 """
-todo = '''SmartAppendSlashMiddleware(object) Not Working. 
+todo = """SmartAppendSlashMiddleware(object) Not Working. 
 It needs re-writing to be compatible with Django v2.0 and later
-'''
+"""
+
+
 class SmartAppendSlashMiddleware(object):
     """
     "SmartAppendSlash" middleware for taking care of URL rewriting.
@@ -20,32 +22,34 @@ class SmartAppendSlashMiddleware(object):
     """
 
     def process_request(self, request):
-        '''Called for every url so return as quickly as possible
+        """Called for every url so return as quickly as possible
         Append a slash if SMART_APPEND_SLASH is set, the resulting URL resolves and it doesn't without the /
-        '''
+        """
         if not settings.SMART_APPEND_SLASH:
             return None
- 
-        if request.path.endswith('/'):
+
+        if request.path.endswith("/"):
             return None
-            
-        if request.path.endswith('_edit'):
+
+        if request.path.endswith("_edit"):
             return None
 
         host = http.HttpRequest.get_host(request)
         old_url = [host, request.path]
         if _resolves(old_url[1]):
             return None
-         
+
         # So: it does not resolve according to our criteria, i.e. _edit doesn't count
-        new_url = old_url[:]       
-        new_url[1] = new_url[1] + '/'
+        new_url = old_url[:]
+        new_url[1] = new_url[1] + "/"
         if not _resolves(new_url[1]):
             return None
-        else:          
-            if settings.DEBUG and request.method == 'POST':
+        else:
+            if settings.DEBUG and request.method == "POST":
                 # replace this exception with a redirect to an error page
-                raise RuntimeError(f"You called this URL via POST, but the URL doesn't end in a slash and you have SMART_APPEND_SLASH set. Django can't redirect to the slash URL while maintaining POST data. Change your form to point to {new_url[0]}{new_url[1]} (note the trailing slash), or set SMART_APPEND_SLASH=False in your Django settings.")
+                raise RuntimeError(
+                    f"You called this URL via POST, but the URL doesn't end in a slash and you have SMART_APPEND_SLASH set. Django can't redirect to the slash URL while maintaining POST data. Change your form to point to {new_url[0]}{new_url[1]} (note the trailing slash), or set SMART_APPEND_SLASH=False in your Django settings."
+                )
         if new_url != old_url:
             # Redirect
             if new_url[0]:
@@ -53,17 +57,18 @@ class SmartAppendSlashMiddleware(object):
             else:
                 newurl = new_url[1]
             if request.GET:
-                newurl += '?' + request.GET.urlencode()
+                newurl += "?" + request.GET.urlencode()
             return http.HttpResponsePermanentRedirect(newurl)
 
         return None
 
+
 def _resolves(url):
     try:
-        # If the URL does not resolve, the function raises a Resolver404 exception (a subclass of Http404) 
+        # If the URL does not resolve, the function raises a Resolver404 exception (a subclass of Http404)
         match = resolve(url)
         # this will ALWAYS be resolved by expopages because it will produce pagenotfound if not the thing asked for
-        # so handle this in expopages, not in middleware 
+        # so handle this in expopages, not in middleware
         return True
     except Resolver404:
         return False