Create base error template

babybuddy/settings/base.py

@@ -77,7 +77,7 @@ ROOT_URLCONF = "babybuddy.urls"
 TEMPLATES = [
     {
         "BACKEND": "django.template.backends.django.DjangoTemplates",
-        "DIRS": [],
+        "DIRS": ["babybuddy/templates/error"],
         "APP_DIRS": True,
         "OPTIONS": {
             "context_processors": [

babybuddy/templates/error/403.html

@@ -1,13 +1,10 @@
-{% extends 'babybuddy/page.html' %}
+{% extends 'error/base.html' %}
-{% load i18n widget_tweaks %}
+{% load i18n %}
 
 {% block title %}403 {% trans "Permission Denied" %}{% endblock %}
 
-{% block breadcrumbs %}
-    <li class="breadcrumb-item active" aria-current="page">{% trans "Permission Denied" %}</li>
-{% endblock %}
-
 {% block content %}
+    <h1>403 {% trans "Permission Denied" %}</h1>
     <div class="alert alert-danger" role="alert">
         {% blocktrans trimmed %}
             You do not have permission to access this resource. Contact a site

babybuddy/templates/error/403_csrf_bad_origin.html

@@ -1,29 +1,21 @@
-{% extends 'babybuddy/base.html' %}
+{% extends 'error/base.html' %}
-{% load i18n widget_tweaks %}
+{% load i18n %}
 
 {% block title %}403 {{ title }}{% endblock %}
 
 {% block breadcrumb_nav %}{% endblock %}
 
-{% block page %}
+{% block content %}
-    <div class="container mt-2 mt-lg-5">
+    <h1>403: {{ title }}</h1>
-        <div class="row justify-content-md-center">
+    <div class="alert alert-danger" role="alert">
-            <div class="col-lg-12 mb-4">
+        {{ main }} {{ reason }}
-                <div>
+    </div>
-                    <h1>403: {{ title }}</h1>
+    <div class="jumbotron">
-                    <div class="alert alert-danger" role="alert">
+        <h2>{% trans "How to Fix" %}</h2>
-                        {{ main }} {{ reason }}
+        {% blocktrans trimmed with origin=origin %}
-                    </div>
+            Add <samp>{{ origin }}</samp> to the <code>CSRF_TRUSTED_ORIGINS</code>
-                    <div class="jumbotron">
+            environment variable. If multiple origins are required separate
-                        <h2>{% trans "How to Fix" %}</h2>
+            with commas.
-                        {% blocktrans trimmed with origin=origin %}
+        {% endblocktrans %}
-                            Add <samp>{{ origin }}</samp> to the <code>CSRF_TRUSTED_ORIGINS</code>
-                            environment variable. If multiple origins are required separate
-                            with commas.
-                        {% endblocktrans %}
-                    </div>
-                </div>
-            </div>
-        </div>
     </div>
 {% endblock %}

babybuddy/templates/error/404.html

@@ -0,0 +1,13 @@
+{% extends 'error/base.html' %}
+{% load i18n %}
+
+{% block title %}404 {% trans "Page Not Found" %}{% endblock %}
+
+{% block content %}
+    <h1>404 {% trans "Page Not Found" %}</h1>
+    <div>
+        {% blocktrans trimmed with request_path=request_path %}
+            The path <samp>{{ request_path }}</samp> does not exist.
+        {% endblocktrans %}
+    </div>
+{% endblock %}

babybuddy/templates/error/base.html

@@ -0,0 +1,14 @@
+{% extends 'babybuddy/base.html' %}
+{% load i18n widget_tweaks %}
+
+{% block breadcrumb_nav %}{% endblock %}
+
+{% block page %}
+    <div class="container mt-2 mt-lg-5">
+        <div class="row justify-content-md-center">
+            <div class="col-lg-12 mb-4">
+                {% block content %}{% endblock %}
+            </div>
+        </div>
+    </div>
+{% endblock %}

babybuddy/views.py

@@ -8,7 +8,7 @@ from django.contrib.messages.views import SuccessMessageMixin
 from django.http import HttpResponseForbidden
 from django.middleware.csrf import REASON_BAD_ORIGIN
 from django.shortcuts import redirect, render
-from django.template import loader, TemplateDoesNotExist
+from django.template import loader
 from django.urls import reverse, reverse_lazy
 from django.utils import translation
 from django.utils.decorators import method_decorator
@@ -34,21 +34,21 @@ def csrf_failure(request, reason=""):
     Overrides the 403 CSRF failure template for bad origins in order to provide more
     userful information about how to resolve the issue.
     """
+
+    print(reason == REASON_BAD_ORIGIN % request.META["HTTP_ORIGIN"])
+
     if (
         "HTTP_ORIGIN" in request.META
         and reason == REASON_BAD_ORIGIN % request.META["HTTP_ORIGIN"]
     ):
-        c = {
+        ccontext = {
             "title": _("Forbidden"),
             "main": _("CSRF verification failed. Request aborted."),
             "reason": reason,
             "origin": request.META["HTTP_ORIGIN"],
         }
-        try:
+        template = loader.get_template("error/403_csrf_bad_origin.html")
-            t = loader.get_template("error/403_csrf_bad_origin.html")
+        return HttpResponseForbidden(template.render(ccontext), content_type="text/html")
-            return HttpResponseForbidden(t.render(c), content_type="text/html")
-        except TemplateDoesNotExist:
-            pass
 
     return csrf.csrf_failure(request, reason, "403_csrf.html")