Skip to content

Commit

Permalink
Return 404 if old/unknown stembureaupages are requested. Show disclai…
Browse files Browse the repository at this point in the history
…mer on embeds
  • Loading branch information
siccovansas committed Mar 7, 2022
1 parent ce44101 commit 75b17a8
Show file tree
Hide file tree
Showing 6 changed files with 33 additions and 31 deletions.
26 changes: 24 additions & 2 deletions app/routes.py
Original file line number Diff line number Diff line change
Expand Up @@ -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],
Expand All @@ -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],
Expand All @@ -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
)


Expand Down
4 changes: 2 additions & 2 deletions app/templates/404.html
Original file line number Diff line number Diff line change
Expand Up @@ -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 %}
4 changes: 2 additions & 2 deletions app/templates/500.html
Original file line number Diff line number Diff line change
Expand Up @@ -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 %}
13 changes: 1 addition & 12 deletions app/templates/embed_stembureau.html
Original file line number Diff line number Diff line change
Expand Up @@ -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>
Expand Down Expand Up @@ -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 }}
Expand Down Expand Up @@ -171,7 +161,6 @@ <h3>Gemeente {{ gemeente }}</h3>
</div>
</div>
{% endif %}
{% endif %}
</div>
</div>
{% endblock %}
Expand Down
4 changes: 3 additions & 1 deletion app/templates/index.html
Original file line number Diff line number Diff line change
Expand Up @@ -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>
Expand Down
13 changes: 1 addition & 12 deletions app/templates/show_stembureau.html
Original file line number Diff line number Diff line change
Expand Up @@ -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>
Expand Down Expand Up @@ -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 }}
Expand Down Expand Up @@ -185,7 +175,6 @@ <h3>Gemeente {{ gemeente }}</h3>
</div>
</div>
</div>
{% endif %}
</div>

<div class="modal fade" tabindex="-1" role="dialog" id="model-embed">
Expand Down

0 comments on commit 75b17a8

Please sign in to comment.