If the gallery view makes sure we have a user anyway, do we need this check?

Seems like the classic annoying "SHOULD NEVER HAPPEN" else: statement :)
This commit is contained in:
Christopher Allan Webber 2011-11-13 14:40:11 -06:00
parent 017d6ca350
commit 7fc25d2720

View File

@ -27,28 +27,23 @@
{% endblock mediagoblin_head %} {% endblock mediagoblin_head %}
{% block mediagoblin_content -%} {% block mediagoblin_content -%}
{% if user %} <h1>
<h1> {%- trans username=user.username,
{%- trans username=user.username, user_url=request.urlgen(
user_url=request.urlgen( 'mediagoblin.user_pages.user_home',
'mediagoblin.user_pages.user_home', user=user.username) -%}
user=user.username) -%} <a href="{{ user_url }}">{{ username }}</a>'s media
<a href="{{ user_url }}">{{ username }}</a>'s media {%- endtrans %}
{%- endtrans %} </h1>
</h1>
<div class="container_16 media_gallery"> <div class="container_16 media_gallery">
{{ object_gallery(request, media_entries, pagination) }} {{ object_gallery(request, media_entries, pagination) }}
</div> </div>
<div class="grid_16"> <div class="grid_16">
{% set feed_url = request.urlgen( {% set feed_url = request.urlgen(
'mediagoblin.user_pages.atom_feed', 'mediagoblin.user_pages.atom_feed',
user=user.username) %} user=user.username) %}
{% include "mediagoblin/utils/feed_link.html" %} {% include "mediagoblin/utils/feed_link.html" %}
</div> </div>
{% else %}
{# This *should* not occur as the view makes sure we pass in a user. #}
<p>{% trans %}Sorry, no such user found.{% endtrans %}<p/>
{% endif %}
{% endblock %} {% endblock %}