aboutsummaryrefslogtreecommitdiffstats
path: root/mediagoblin/templates
diff options
context:
space:
mode:
authorSebastian Spaeth <Sebastian@SSpaeth.de>2011-05-11 08:39:57 +0200
committerChristopher Allan Webber <cwebber@dustycloud.org>2011-05-12 23:10:21 -0500
commit7acdbfd36457720789c183260424e0f78139211a (patch)
tree32d41524cefc9103cf7f5f55fda824f7dbdbb518 /mediagoblin/templates
parentb77eec653df14059296fc3185ff9817edfa0825b (diff)
downloadmediagoblin-7acdbfd36457720789c183260424e0f78139211a.tar.lz
mediagoblin-7acdbfd36457720789c183260424e0f78139211a.tar.xz
mediagoblin-7acdbfd36457720789c183260424e0f78139211a.zip
Improve user homepage
1) Only consider user's with 'status': 'active'. We don't want to display unconfirmed/blocked users, right? 2) Actually query user's media in the view and display on their home page. 3) Throw an error 404 if we don't find a valid user, rather than saying, "User not found" (from within the template). 4) Pass in medias as media_entries to remain compatible with the 'root' page. Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
Diffstat (limited to 'mediagoblin/templates')
-rw-r--r--mediagoblin/templates/mediagoblin/user_pages/user.html15
1 files changed, 14 insertions, 1 deletions
diff --git a/mediagoblin/templates/mediagoblin/user_pages/user.html b/mediagoblin/templates/mediagoblin/user_pages/user.html
index 4fa84430..85f05e08 100644
--- a/mediagoblin/templates/mediagoblin/user_pages/user.html
+++ b/mediagoblin/templates/mediagoblin/user_pages/user.html
@@ -19,8 +19,21 @@
{% block mediagoblin_content -%}
{% if user %}
<h2>User page for '{{ user.username }}'</h2>
- {{ user }}
+
+ {#- Should we outsource such a media 'gallery' view to it's own file?
+ It could be useful for the home page and other views too -#}
+ <ul>
+ {%- for entry in media_entries %}
+ <li>
+ <a href="{{ request.urlgen('mediagoblin.user_pages.media_home',
+ user= entry.uploader.username, m_id= entry._id) }}">
+ <img src="{{ request.app.public_store.file_url(
+ entry['media_files']['thumb']) }}" /></a>
+ </li>
+ {%- endfor %}
+ </ul>
{% else %}
+ {# This *should* not occur as the view makes sure we pass in a user. #}
<p>Sorry, no such user found.<p/>
{% endif %}
{% endblock %}