[#21] refactor context processor

open_api_framework/conf/base.py

@@ -181,7 +181,7 @@
                 "django.template.context_processors.request",
                 "django.contrib.auth.context_processors.auth",
                 "django.contrib.messages.context_processors.messages",
-                "open_api_framework.context_processors.project",
+                "open_api_framework.context_processors.admin_settings",
                 f"{PROJECT_DIRNAME}.utils.context_processors.settings",
             ],
             "loaders": TEMPLATE_LOADERS,

open_api_framework/context_processors.py

@@ -1,11 +1,30 @@
 from django.conf import settings
+from django.http.request import HttpRequest
 
 
-def project(request):
-    django_settings = (
-        "ENVIRONMENT_SHOWN_IN_ADMIN",
-        "RELEASE",
-        "GIT_SHA",
-    )
+def admin_settings(request: HttpRequest) -> dict:
+    resolver = request.resolver_match
+    show_version = False
 
-    return {"project": {k: getattr(settings, k, None) for k in django_settings}}
+    if resolver:
+        if resolver.app_name:
+            url_name = f"{resolver.app_name}:{resolver.url_name}"
+        else:
+            url_name = resolver.url_name
+
+        # Note that only views that use the open-api-framework's
+        # `base_site.html` template are listed here
+        excluded_version_urls = [
+            "admin_password_reset",
+            "password_reset_done",
+        ]
+
+        show_version = bool(url_name not in excluded_version_urls)
+
+    return {
+        "show_environment": getattr(settings, "ENVIRONMENT_SHOWN_IN_ADMIN", None),
+        "show_version": show_version,
+        "git_sha": getattr(settings, "GIT_SHA", None),
+        "version": getattr(settings, "RELEASE", None),
+        "environment": getattr(settings, "ENVIRONMENT", None),
+    }

open_api_framework/templates/open_api_framework/admin/base_site.html

@@ -6,18 +6,16 @@
 {% endblock %}
 
 {% block messages %}
-    {% if project.ENVIRONMENT_SHOWN_IN_ADMIN %}
+    {% if show_environment %}
         {% environment %}
     {% endif %}
 
     {{ block.super }}
 {% endblock %}
 
 {% block footer %}
-    {% url 'admin:login' as admin_login_url %}
-
     <div id="footer">
-        {% if request.path != admin_login_url %}
+        {% if show_version %}
             {% version %}
         {% endif %}
     </div>