From 75b17a8b1512a66b5e24201c42932cfc62ba18d2 Mon Sep 17 00:00:00 2001
From: Sicco van Sas <siccovansas@users.noreply.github.com>
Date: Mon, 7 Mar 2022 12:16:48 +0100
Subject: [PATCH] Return 404 if old/unknown stembureaupages are requested. Show
 disclaimer on embeds

---
 app/routes.py                       | 26 ++++++++++++++++++++++++--
 app/templates/404.html              |  4 ++--
 app/templates/500.html              |  4 ++--
 app/templates/embed_stembureau.html | 13 +------------
 app/templates/index.html            |  4 +++-
 app/templates/show_stembureau.html  | 13 +------------
 6 files changed, 33 insertions(+), 31 deletions(-)

diff --git a/app/routes.py b/app/routes.py
index 991009d..b163611 100644
--- a/app/routes.py
+++ b/app/routes.py
@@ -211,6 +211,10 @@ def show_stembureau(gemeente, primary_key):
     records = get_stembureaus(
         ckan.elections, {'Gemeente': gemeente, 'UUID': primary_key}
     )
+
+    if not records:
+        return render_template('404.html'), 404
+
     return render_template(
         'show_stembureau.html',
         records=[_hydrate(record, 'extended') for record in records],
@@ -227,6 +231,7 @@ def show_gemeente(gemeente):
         disclaimer = disclaimer_text
 
     records = get_stembureaus(ckan.elections, {'Gemeente': gemeente})
+
     return render_template(
         'show_gemeente.html',
         records=[_hydrate(record, 'default') for record in records],
@@ -237,28 +242,45 @@ def show_gemeente(gemeente):
 
 @app.route("/e/<gemeente>/<primary_key>")
 def embed_stembureau(gemeente, primary_key):
+    disclaimer = ''
+    if gemeente in disclaimer_gemeenten:
+        disclaimer = disclaimer_text
+
     records = get_stembureaus(
         ckan.elections, {'Gemeente': gemeente, 'UUID': primary_key}
     )
+
+    if not records:
+        return render_template('404.html'), 404
+
     show_infobar = (request.args.get('infobar', 1, type=int) == 1)
+
     return render_template(
         'embed_stembureau.html',
         records=[_hydrate(record, 'extended') for record in records],
         gemeente=gemeente,
         primary_key=primary_key,
-        show_infobar=show_infobar
+        show_infobar=show_infobar,
+        disclaimer=disclaimer
     )
 
 
 @app.route("/e/<gemeente>")
 def embed_gemeente(gemeente):
+    disclaimer = ''
+    if gemeente in disclaimer_gemeenten:
+        disclaimer = disclaimer_text
+
     records = get_stembureaus(ckan.elections, {'Gemeente': gemeente})
+
     show_search = (request.args.get('search', 1, type=int) == 1)
+
     return render_template(
         'embed_gemeente.html',
         records=[_hydrate(record, 'default') for record in records],
         gemeente=gemeente,
-        show_search=show_search
+        show_search=show_search,
+        disclaimer=disclaimer
     )
 
 
diff --git a/app/templates/404.html b/app/templates/404.html
index dd4bcc4..f81044c 100644
--- a/app/templates/404.html
+++ b/app/templates/404.html
@@ -7,7 +7,7 @@
 
 {% block content %}
   <div class="container">
-    <h1 class="text-center">Bestand niet gevonden</h1>
-    <p><a href="{{ url_for('index') }}">Naar de homepage</a></p>
+    <h1 class="text-center">Pagina niet gevonden</h1>
+    <p class="text-center"><a href="{{ url_for('index') }}">Naar de homepage</a></p>
   </div>
 {% endblock %}
diff --git a/app/templates/500.html b/app/templates/500.html
index 9c764ac..de93c5b 100644
--- a/app/templates/500.html
+++ b/app/templates/500.html
@@ -8,7 +8,7 @@
 {% block content %}
   <div class="container">
     <h1 class="text-center">Er is een onverwachte fout opgetreden</h1>
-    <p>De websitebeheerder is op de hoogte gesteld. Excuses voor het ongemak!</p>
-    <p><a href="{{ url_for('index') }}">Naar de homepage</a></p>
+    <p class="text-center">De websitebeheerder is op de hoogte gesteld. Excuses voor het ongemak!</p>
+    <p class="text-center"><a href="{{ url_for('index') }}">Naar de homepage</a></p>
   </div>
 {% endblock %}
diff --git a/app/templates/embed_stembureau.html b/app/templates/embed_stembureau.html
index d7cf3bf..882b23d 100644
--- a/app/templates/embed_stembureau.html
+++ b/app/templates/embed_stembureau.html
@@ -6,11 +6,7 @@
       <ol class="breadcrumb">
         <li><a href="/" target="_blank" rel="noopener">Nederland</a></li>
         <li><a href="{{ url_for('show_gemeente', gemeente=gemeente) }}" target="_blank" rel="noopener">{{ gemeente }}</a></li>
-        {% if not records %}
-          <li class="active">Onbekend stembureau UUID</li>
-        {% else %}
-          <li class="active">{{ records[0]['Naam stembureau'] }}</li>
-        {% endif %}
+        <li class="active">{{ records[0]['Naam stembureau'] }}</li>
       </ol>
     </div>
   </div>
@@ -39,12 +35,6 @@ <h1>
   </div>
 
   <div class="row">
-    {% if not records %}
-    Kan stembureau met UUID "{{ primary_key }}" niet vinden.
-    <br>
-    <br>
-    Terug naar de <a href="{{ url_for('embed_gemeente', gemeente=gemeente) }}">stembureaupagina van gemeente {{ gemeente }}</a>.
-    {% else %}
     {% if disclaimer %}
     <div class="col-xs-12">
       {{ disclaimer }}
@@ -171,7 +161,6 @@ <h3>Gemeente {{ gemeente }}</h3>
       </div>
     </div>
     {% endif %}
-    {% endif %}
   </div>
 </div>
 {% endblock %}
diff --git a/app/templates/index.html b/app/templates/index.html
index 8652004..126dc1f 100644
--- a/app/templates/index.html
+++ b/app/templates/index.html
@@ -8,7 +8,9 @@
 
 {% block content %}
   <div class="container-fluid">
-    <span style="color: red">De informatie over alle stembureaus voor de gemeenteraadsverkiezingen 2022 is nog niet compleet en wordt tot en met de verkiezingen continu aangevuld</span>
+    <div class="text-center">
+      <span style="color: red">De informatie over alle stembureaus voor de gemeenteraadsverkiezingen 2022 is nog niet compleet en wordt tot en met de verkiezingen continu aangevuld</span>
+    </div>
     {% include "show_alles.html" %}
     {% include "search_gemeenten.html" %}
   </div>
diff --git a/app/templates/show_stembureau.html b/app/templates/show_stembureau.html
index c3f0dce..4755ea3 100644
--- a/app/templates/show_stembureau.html
+++ b/app/templates/show_stembureau.html
@@ -17,11 +17,7 @@
       <ol class="breadcrumb">
         <li><a href="/">Nederland</a></li>
         <li><a href="{{ url_for('show_gemeente', gemeente=gemeente) }}">{{ gemeente }}</a></li>
-        {% if not records %}
-          <li class="active">Onbekend stembureau UUID</li>
-        {% else %}
-          <li class="active">{{ records[0]['Naam stembureau'] }}</li>
-        {% endif %}
+        <li class="active">{{ records[0]['Naam stembureau'] }}</li>
       </ol>
     </div>
   </div>
@@ -50,12 +46,6 @@ <h1>
   </div>
 
   <div class="row">
-    {% if not records %}
-    Kan stembureau met UUID "{{ primary_key }}" niet vinden.
-    <br>
-    <br>
-    Terug naar de <a href="{{ url_for('show_gemeente', gemeente=gemeente) }}">stembureaupagina van gemeente {{ gemeente }}</a>.
-    {% else %}
     {% if disclaimer %}
     <div class="col-xs-12">
       {{ disclaimer }}
@@ -185,7 +175,6 @@ <h3>Gemeente {{ gemeente }}</h3>
         </div>
       </div>
     </div>
-    {% endif %}
   </div>
 
   <div class="modal fade" tabindex="-1" role="dialog" id="model-embed">